Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not set empty structures in request to create aws integration #505

Merged
merged 1 commit into from May 6, 2020
Merged

Do not set empty structures in request to create aws integration #505

merged 1 commit into from May 6, 2020

Conversation

miguelaferreira
Copy link
Contributor

This PR Fixes #504 by only setting host tags, filter tags and account specific namespace rules on the http request if these values have been set by the user.

Somehow, setting these structures while they are empty affects the serialisation of the request in such a way that the DataDog API rejects the values as being of the wrong type.

@ghost ghost added the size/XS label May 4, 2020
Copy link
Contributor

@gzussa gzussa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for this! 💯

@gzussa gzussa merged commit 9f968ec into DataDog:master May 6, 2020
@miguelaferreira
Copy link
Contributor Author

Thanks for taking the time @gzussa. Do you know when can we expect a new release?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error regarding host_tags while using datadog_integration_aws
2 participants