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

Allow headers size extend to maxRequestHeadersSize in http client #12438

Closed
wants to merge 7 commits into from

Conversation

shaoxt
Copy link
Contributor

@shaoxt shaoxt commented Oct 29, 2024

PR for #12436

@sbordet
Copy link
Contributor

sbordet commented Oct 29, 2024

@shaoxt make sure you comply with the Eclipse ECA, see https://github.com/jetty/jetty.project/blob/jetty-12.0.x/CONTRIBUTING.md, otherwise we cannot accept this contribution.

@shaoxt
Copy link
Contributor Author

shaoxt commented Oct 29, 2024

Screenshot 2024-10-29 at 1 03 30 PM

@shaoxt
Copy link
Contributor Author

shaoxt commented Oct 29, 2024

Had PR merged before. And resigned the ECA again.

@shaoxt make sure you comply with the Eclipse ECA, see https://github.com/jetty/jetty.project/blob/jetty-12.0.x/CONTRIBUTING.md, otherwise we cannot accept this contribution.

@joakime
Copy link
Contributor

joakime commented Oct 29, 2024

Your various commits on this PR have 3 different email addresses associated with it.

Example:

Squash (or rebase) all of these commits and make it a single commit with just your registered gmail address.

@shaoxt
Copy link
Contributor Author

shaoxt commented Oct 29, 2024

Close this PR for the invalid authors.

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.

3 participants