hashicorp / hashicorp/consul

Allow configurable max_request_header_size

Open
#20,992 0 comments 2 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
30.1k
Forks
4.6k
Avg merge
2d 6h
Merged PRs (30d)
43

Description

### Community Note

* Please vote on this issue by adding a 👍 [reaction](https://blog.github.com/2016-03-10-add-reactions-to-pull-requests-issues-and-comments/) to the original issue to help the community and maintainers prioritize this request. Searching for pre-existing feature requests helps us consolidate datapoints for identical requirements into a single place, thank you!
* Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.
* If you are interested in working on this issue or have submitted a pull request, please leave a comment.

---

#### Is your feature request related to a problem? Please describe.

We have a multi-tenancy backend where identities and roles are embedded inside the request header. Default Envoy's HTTPConnectionManager header size is 60KB but we need a lot more than that. However, after tinkering with escape-hatch configurations with custom listeners and cluster, we couldn't figure out what was needed to make it work.

#### Feature Description

Allow configurable `max_request_header_size` [(defined here)](https://www.envoyproxy.io/docs/envoy/latest/api-v3/extensions/filters/network/http_connection_manager/v3/http_connection_manager.proto#:~:text=scheme%20handling%20applies.-,max_request_headers_kb,-(UInt32Value)%20The) as part of the ServiceDefault or ProxyDefault manifest

#### Use Case(s)

Systems that use a lot of headers

#### Contributions

After tinkering with Consul for a bit, i found that `command/connect/envoy/bootstrap_config.go` contains the template for such listener. Happy to contribute if I can get a confirmation that that's where I should be editting though.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.