TF_LOG=trace terraform apply does not print request logs

[morganchristiansson]

When running TF_LOG=trace terraform apply requests logs are not printed.

When doing the same when running acceptance tests they are however.

I had a look in source code and newrelic-go dependency and it seems like it should be output when TF_LOG=trace is set

Terraform Version

Terraform v0.12.12

• provider.newrelic v1.5.2

Affected Resource(s)

provider.newrelic

Expected Behavior

HTTP request and responses be output to console to view.

Actual Behavior

HTTP requests are not in output. And because requests are using https it's the requests are not easily available for debug.

Steps to Reproduce

1. Create any newrelic resource
2. Run TF_LOG=trace terraform apply

[morganchristiansson]

newrelic.config.Debug is set to logging.IsDebugOrHigher()
https://github.com/terraform-providers/terraform-provider-newrelic/blob/bb6d85381287ce74ba9740a553471bf9f2ec03e3/newrelic/config.go#L21

if config.Debug is true then RestyClient r.setDebug(true)
https://github.com/paultyng/go-newrelic/blob/1a50743299616d5a9ab9f30848121f2df9023e55/api/client.go#L94

So RestyClient r.setDebug(true) is called.

Yet request logs are not appearing

[ctrombley]

@morganchristiansson what you're experiencing is due to the issue described in this thread:
hashicorp/terraform-plugin-sdk#88 (comment)

Terraform core and the providers are separate processes communicating via RPC, and the logs from the SDK are not making their way through to the console when using the terraform command.

I have personally had luck using the Charles web proxy as described in the thread above to inspect at the HTTP layer.

[morganchristiansson]

Ok makes sense. However the pagerduty provider was printing http requests successfully in the same run.. haven't looked at why.

[ctrombley]

Here's why! Looks like this is achievable after all. I will put a quick PR in.

[ctrombley]

Blocked on merging paultyng/go-newrelic#76.

[ctrombley]

Targeting v1.7.0 for a fix.

[sanderblue]

Fixed in #228 per paultyng/go-newrelic#76