I noticed that CLOUDAMQP_URL is used a lot in our documentation for holding the AMQP connection string. To avoid conflicts with the CLI and our docs example code I propose a change in client/client.go from CLOUDAMQP_URL to CLOUDAMQP_API_URL or CLOUDAMQP_API_BASE_URL
|
if envURL := os.Getenv("CLOUDAMQP_URL"); envURL != "" { |
The CLI is not satisfied with CLOUDAMQP_URL emptied and will attempt to use the AMQP connection string if set.
Happy to create a PR if this is accepted or other way to handle the env conflicts.
I noticed that CLOUDAMQP_URL is used a lot in our documentation for holding the AMQP connection string. To avoid conflicts with the CLI and our docs example code I propose a change in client/client.go from
CLOUDAMQP_URLtoCLOUDAMQP_API_URLorCLOUDAMQP_API_BASE_URLcli/client/client.go
Line 25 in 5458022
The CLI is not satisfied with
CLOUDAMQP_URLemptied and will attempt to use the AMQP connection string if set.Happy to create a PR if this is accepted or other way to handle the env conflicts.