Add param to forward headers from the auth server to the origin #9271
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Ftywan
force-pushed
the
auth-proxy-header-forward
branch
2 times, most recently
from
aebcb08 to
446c95a
Compare
|
[approve ci] |
|
Can we add this option to the documentation? Will all headers that match the prefix that come back from the auth server be proxied to the origin? What's the format of the value (just text)? Is there just one prefix that can be configured? (think the doc is at doc/admin-guide/plugins/authproxy.en.rst) |
Ftywan
force-pushed
the
auth-proxy-header-forward
branch
from
446c95a to
b93ee5d
Compare
|
Hey @mlibbey, I've added the option usage to the documentation. |
Ftywan
force-pushed
the
auth-proxy-header-forward
branch
2 times, most recently
from
a94f7bb to
7fb41e4
Compare
Ftywan
force-pushed
the
auth-proxy-header-forward
branch
from
7fb41e4 to
f57e897
Compare
bryancall
approved these changes
Jan 19, 2023
masaori335
added a commit
to masaori335/trafficserver
that referenced
this pull request
Jul 24, 2023
…he#9271) (apache#482) (cherry picked from commit b7b25f7) Co-authored-by: Fu Tianyuan <ftygary@gmail.com>
cmcfarlen
pushed a commit
to cmcfarlen/trafficserver
that referenced
this pull request
Jun 3, 2024
* asf/master: (623 commits) records.config to records.yaml (apache#9264) Updates the release roadmap, adjusting for delays (apache#9360) Upgrades master branch to use clang-format v15.0.7 (apache#9355) Disable merging on GitHub (apache#9354) Clang-format 15.0.7 is finicky, and does not like these old school array inits (apache#9356) Enable merging for 10-Dev merge (apache#9353) Fix an error on SSL config reload (plus some cleanup). (apache#9334) Cleanup of legacy, makes newer clang-format crash right now (apache#9350) Update the roadmap / branch management doc page (apache#9340) Proxy Protocol out fixes (apache#9341) Memory leaks with storing configuration filenames (apache#9324) s3_auth autest: convert from gold file to file contains (apache#9337) Make 204 cacheable again (apache#9333) Add param to forward headers from the auth server to the origin (apache#9271) s3_auth: Fix assertion failure of TSActionCancel (apache#9329) Added http connect Autest with proxy verifier (apache#9315) s3_auth: Schedule reloading config event on TASK thread (apache#9328) Register ET_UDP thread type even if no UDP threads are requested (apache#9314) Don't send response body on status 204 No Content (apache#9330) Limit the serching range of static table by the first letter of header name (apache#9298) ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added a header prefix field to the config. Headers from the authentication response with the prefix will be appended to the original request.
new parameter:
@pparam=--forward-header-prefix