Skip to content

Add optional query param for adding headers to the server's response#146

Open
moritzschmitz-oviva wants to merge 25 commits intotraefik:masterfrom
moritzschmitz-oviva:feature/add-response-header-param
Open

Add optional query param for adding headers to the server's response#146
moritzschmitz-oviva wants to merge 25 commits intotraefik:masterfrom
moritzschmitz-oviva:feature/add-response-header-param

Conversation

@moritzschmitz-oviva
Copy link

@moritzschmitz-oviva moritzschmitz-oviva commented Apr 9, 2025

Add a feature to allow setting response headers via query params.

@ldez
Copy link
Contributor

ldez commented Apr 9, 2025

Note, this doesn't fix #31 because this issue was not related to query parameters but to a request without any arguments.
The issue was about CLI flags.

@moritzschmitz-oviva moritzschmitz-oviva requested a review from ldez April 9, 2025 16:39
moritzschmitz-oviva and others added 2 commits April 9, 2025 20:46
Co-authored-by: Ludovic Fernandez <ldez@users.noreply.github.com>
@moritzschmitz-oviva
Copy link
Author

@ldez Thank you for the input!

@emilevauge emilevauge removed the request for review from ldez May 22, 2025 13:27
@mmatur mmatur force-pushed the feature/add-response-header-param branch from fc653de to bf1aefd Compare May 22, 2025 13:30
@moritzschmitz-oviva
Copy link
Author

@mmatur I think your review got stale?

@moritzschmitz-oviva
Copy link
Author

@ldez @mmatur Anything I can do for this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants