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

Consider methods to consolidate headers in nginx configuration #440

Open
phantomjinx opened this issue Apr 18, 2024 · 0 comments
Open

Consider methods to consolidate headers in nginx configuration #440

phantomjinx opened this issue Apr 18, 2024 · 0 comments
Milestone

Comments

@phantomjinx
Copy link
Member

To conform with security requirements, the number of add_header config lines is proliferating in the nginx config template. One of the reasons for the number is that adding a single header in a location will not merge with the parent's set of headers but overwrite them!

Therefore, all parent headers must be specified again in a child location if the latter has an additional add_header entry.

There are workarounds to this that could be considered:

  • most_set_headers directive - provides an alternative merge-style directive to be used instead of add_header;
  • include directive - putting all the add_header entries into a single file then include that file with a single-line include directive.
@phantomjinx phantomjinx added this to the 2024Q2 milestone Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

No branches or pull requests

1 participant