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

feat(h2): add max_header_list_size #2828

Merged
merged 2 commits into from
May 18, 2022
Merged

Conversation

silence-coding
Copy link
Contributor

fix #2826

@silence-coding
Copy link
Contributor Author

@seanmonstar Please check

@lidong14
Copy link

lidong14 commented May 7, 2022

@seanmonstar when will publish new version with this issue ? i need ,ths

@lidong14
Copy link

@seanmonstar When will you merge this issue ? ths

@lidong14
Copy link

@seanmonstar do you review the merge ? when will be merged?

src/server/conn.rs Outdated Show resolved Hide resolved
…tp2_max_header_list_size

Co-authored-by: Sean McArthur <[email protected]>
@silence-coding
Copy link
Contributor Author

@seanmonstar Thanks

@silence-coding
Copy link
Contributor Author

I don't know what this error has to do with my code.
image

@lidong14
Copy link

@seanmonstar please review the merge , thanks

@seanmonstar seanmonstar merged commit a32658c into hyperium:master May 18, 2022
@joseluisq
Copy link

joseluisq commented May 28, 2022

Correct me if I'm wrong but should not be max_header_list_size also replicated to hyper::server::Builder?

@AshleySchaeffer
Copy link

Hate doing this but I think I need this; bump regarding adding this to hyper::server::Builder.

jiahaoliang added a commit to jiahaoliang/hyper that referenced this pull request Oct 8, 2022
jiahaoliang added a commit to jiahaoliang/hyper that referenced this pull request Dec 1, 2022
seanmonstar pushed a commit that referenced this pull request Dec 20, 2022
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.

Allow specifying the HTTP/2 SETTINGS_MAX_HEADER_LIST_SIZE
5 participants