Skip to content

/ caddy Public

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

Jump to bottom

Add support for wildcards/ expression matching in header_up/ header_down #3913

Open
gucki opened this issue Dec 12, 2020 · 3 comments
Open

Add support for wildcards/ expression matching in header_up/ header_down #3913

gucki opened this issue Dec 12, 2020 · 3 comments
Labels
feature request good first issue
Milestone
2.x

Comments

@gucki
Copy link

@gucki gucki commented Dec 12, 2020

It seems Caddy does not support wildcards/ expression matching in the reverse proxy's header_up.

For example this does not remove any headers:

header_up -X-SHIBBOLETH-*

This works as expected, but is limited in its use:

header_up -X-SHIBBOLETH-LOGIN

It'd be great if Caddy would support wildcards/ expression matching in the reverse proxy's header_up/ header_down.
@gucki gucki changed the title Support wildcards/ expression matching in header_up Add support for wildcards/ expression matching in header_up/ header_down Dec 12, 2020
@mholt mholt added this to the 2.x milestone Dec 12, 2020
@rajat315315 rajat315315 mentioned this issue Jan 25, 2021
Closed
@simaotwx
Copy link
Contributor

@simaotwx simaotwx commented Mar 19, 2021

See #3992 (comment) for my suggestion

@Mohit-Pawar
Copy link

@Mohit-Pawar Mohit-Pawar commented Jul 23, 2021

@mholt I would like to work on some good first issue. can i take this up?

@francislavoie
Copy link
Member

@francislavoie francislavoie commented Jul 23, 2021

Sure @Mohit-Pawar, just be aware of the prior work in #3992

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request good first issue
Projects
None yet
Milestone
2.x
Linked pull requests

Successfully merging a pull request may close this issue.

Added regex pattern matching for Deleting Headers
5 participants
@gucki @mholt @francislavoie @Mohit-Pawar @simaotwx