Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

server: allow setting max_header_list_size #1870

Merged
merged 2 commits into from
Aug 20, 2024

Conversation

BugenZhao
Copy link
Contributor

Signed-off-by: Bugen Zhao i@bugenzhao.com

Motivation

This is basically the dual of #1835 on the server side. Can refer to the motivation of #1835.

Solution

Add the configuration for the server with the same interface as the client (transport). Also refine the documentation.

Signed-off-by: Bugen Zhao <i@bugenzhao.com>
Signed-off-by: Bugen Zhao <i@bugenzhao.com>
Copy link
Collaborator

@tottoto tottoto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds reasonable to me.

@djc djc enabled auto-merge August 20, 2024 10:41
@djc djc added this pull request to the merge queue Aug 20, 2024
Merged via the queue into hyperium:master with commit 086bcd2 Aug 20, 2024
16 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants