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

Set request.Close in httpdriver #925

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Set request.Close in httpdriver #925

wants to merge 1 commit into from

Conversation

tonyhb
Copy link
Contributor

@tonyhb tonyhb commented Dec 7, 2023

request.GetBody could be used in the rare circumstance that an external SDK's ALB redirects to another source while part of the body has been consumed. In this circumstance, we give the Golang HTTP client a method to retrieve a fresh copy of the request body for the next HTTP request.

Type of change (choose one)

  • Chore (refactors, upgrades, etc.)
  • Bug fix (non-breaking change that fixes an issue)
  • Security fix (non-breaking change that fixes a potential vulnerability)
  • Docs
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality not to work as expected)

Checklist

  • I've linked any associated issues to this PR.
  • I've tested my own changes.

Check our Pull Request Guidelines

Copy link

vercel bot commented Dec 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
ui ⬜️ Ignored (Inspect) Visit Preview Dec 11, 2023 10:50pm

@tonyhb tonyhb changed the title Set request.GetBody and request.Close in httpdriver Set request.Close in httpdriver Dec 11, 2023
@darwin67
Copy link
Collaborator

darwin67 commented Jan 3, 2024

@tonyhb is this PR still needed?

@tonyhb
Copy link
Contributor Author

tonyhb commented Jan 5, 2024

@tonyhb is this PR still needed?

If anything it's a (very minor) net positive, so I'll rebase and merge later.

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

Successfully merging this pull request may close these issues.

None yet

2 participants