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

Retry policy will always clone the *http.Request #20843

Merged
merged 2 commits into from
May 12, 2023

Conversation

jhendrixMSFT
Copy link
Member

This ensures that the entirety of the request is restored on retries.

  • The purpose of this PR is explained in this or a referenced issue.
  • The PR does not update generated files.
  • Tests are included and/or updated for code changes.
  • Updates to CHANGELOG.md are included.
  • MIT license headers are included in each file.

This ensures that the entirety of the request is restored on retries.
@jhendrixMSFT jhendrixMSFT merged commit a3b2c13 into Azure:main May 12, 2023
@jhendrixMSFT jhendrixMSFT deleted the azcore_retry_clone branch May 12, 2023 15:17
jhendrixMSFT added a commit to jhendrixMSFT/azure-sdk-for-go that referenced this pull request Jun 5, 2023
* Retry policy will always clone the *http.Request

This ensures that the entirety of the request is restored on retries.

* simplify test
jhendrixMSFT added a commit that referenced this pull request Jun 5, 2023
* Retry policy will always clone the *http.Request (#20843)

* Retry policy will always clone the *http.Request

This ensures that the entirety of the request is restored on retries.

* simplify test

* Handle more error codes when an RP isn't registered (#20848)

There's more than one error code returned from unregistered RPs.

* Add another non-standard error code for RP registration (#20860)

* Prep azcore v1.6.1 for release

Cherry-picked the following commits.
- a3b2c13
- 889be58
- 6879d7e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants