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

[PR #5927/33df7b61 backport][stable-6] Set User-Agent for API requests to DNSimple #6005

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Feb 17, 2023

This is a backport of PR #5927 as merged into main (33df7b6).

SUMMARY

What is the problem?

At present the default user agent is used in API requests: dnsimple-python/*. This makes it impossible for us at DNSimple to identify active community projects that are using our API, and how to best support them.

What has changed

With this PR the user agent is changed to: ansible dnsimple-python/* which serves to both identify API usage from the project and also give us debugging data to assist with customer support requests.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

dnsimple

ADDITIONAL INFORMATION

* Set the user-agent for API requests to DNSimple

* Update user agent format

* Add changelog fragment

* Update changelogs/fragments/5927-set-user-agent-dnsimple.yml

Co-authored-by: Felix Fontein <[email protected]>

---------

Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit 33df7b6)
@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added backport docs module module needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR net_tools new_contributor Help guide this first time contributor plugins plugin (any type) and removed needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Feb 17, 2023
@felixfontein felixfontein merged commit 00d1160 into stable-6 Feb 17, 2023
@felixfontein felixfontein deleted the patchback/backports/stable-6/33df7b61c008ee3331b27af2b43d8c90e04f0c9c/pr-5927 branch February 17, 2023 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs module module net_tools new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants