-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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 multiple secure middlewares to operate independently #6604
Conversation
Moving to needs-review as requested by @rtribotte |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Can we get this back ported to 2.2.X? It seems like 2.3.0 is still a bit away and I would really like to get this bug fix to not duplicated my headers 50 times. |
This PR was merged in the wrong branch, it was a mistake, we already back ported this PR inside another PR: https://github.com/containous/traefik/pull/7230/commits We will release a v2.2.11 tomorrow. |
What does this PR do?
Utilizes the secureContextKey to allow different header middlewares to operate independently, without overwriting the next middleware's context key.
Motivation
Fixes #5538
More
Additional Notes
Because
unrolled/secure
uses a private secureContextKey type for storage, we cannot retrieve from this project. Testing this drills down to the base secure project, which is already tested in their repo.Therefore integration testing I feel is the best option at this point.