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

(feature) Disable version header in relay proxy #3140

Closed
tomflenner opened this issue Feb 24, 2025 · 4 comments · Fixed by #3141
Closed

(feature) Disable version header in relay proxy #3140

tomflenner opened this issue Feb 24, 2025 · 4 comments · Fixed by #3141
Assignees
Labels
📍 Assigned enhancement New feature or request p2 Medium priority

Comments

@tomflenner
Copy link
Contributor

Requirements

Hello !

We're using RelayProxy in production since several month now and our OPS teams reported that the x-gofeatureflag-version header was forward in HTTP requests. It would be nice to not have to tweak our Kong gateway to remove this header and let the component himself do it. Our team don't want to expose infrastructure concern inside the HTTP request for some reason.

After discussing with @thomaspoignant, this header is used to detect GOFF version and disable some feature in the provider for compatibility reason. At the moment he never needed to use this feature.

I suggest to allow users to deactivate this header from configuration to be more flexible on relay proxy usage.

@tomflenner tomflenner added enhancement New feature or request needs-triage A priority should be added to the issue labels Feb 24, 2025
@tomflenner
Copy link
Contributor Author

/assign-me

Copy link
Contributor

👋 Hey @tomflenner, this issue is already assigned to @thomaspoignant.

⚠️ It will become unassigned if it isn't closed within 20 days.

🔧 A maintainer can also add you to the list of assignees or swap you with the current assignee.

@thomaspoignant thomaspoignant removed their assignment Feb 24, 2025
@thomaspoignant thomaspoignant added p2 Medium priority and removed needs-triage A priority should be added to the issue labels Feb 24, 2025
@tomflenner
Copy link
Contributor Author

/assign-me

Copy link
Contributor

👋 Hey @tomflenner, thanks for your interest in this issue! 🎉

⚠ Note that this issue will become unassigned if it isn't closed within 20 days.

🔧 A maintainer can also add the 📌 Pinned label to prevent it from being unassigned automatically.

tomflenner pushed a commit to tomflenner/go-feature-flag that referenced this issue Feb 24, 2025
add a boolean EnableVersionHeader in configuration to conditionnally register the VersionHeader custom middleware and disable the header x-gofeatureflag-version

issue: thomaspoignant#3140
tomflenner pushed a commit to tomflenner/go-feature-flag that referenced this issue Feb 24, 2025
add a boolean EnableVersionHeader in configuration to conditionnally register the VersionHeader custom middleware and disable the header x-gofeatureflag-version

issue: thomaspoignant#3140
tomflenner pushed a commit to tomflenner/go-feature-flag that referenced this issue Feb 24, 2025
add a boolean EnableVersionHeader in configuration to conditionnally register the VersionHeader custom middleware and disable the header x-gofeatureflag-version

issue: thomaspoignant#3140
tomflenner pushed a commit to tomflenner/go-feature-flag that referenced this issue Feb 24, 2025
add a boolean DisableVersionHeader in configuration with Skipper on the VersionHeader middleware to

issue: thomaspoignant#3140
tomflenner pushed a commit to tomflenner/go-feature-flag that referenced this issue Feb 24, 2025
add a boolean EnableVersionHeader in configuration to conditionnally register the VersionHeader custom middleware and disable the header x-gofeatureflag-version

issue: thomaspoignant#3140
tomflenner pushed a commit to tomflenner/go-feature-flag that referenced this issue Feb 24, 2025
add a boolean DisableVersionHeader in configuration with Skipper on the VersionHeader middleware to

issue: thomaspoignant#3140
kodiakhq bot pushed a commit that referenced this issue Feb 25, 2025
* feat(relayproxy): allow disable VersionHeader middleware

add a boolean EnableVersionHeader in configuration to conditionnally register the VersionHeader custom middleware and disable the header x-gofeatureflag-version

issue: #3140

* feat(relayproxy): allow disable VersionHeader middleware

add a boolean DisableVersionHeader in configuration with Skipper on the VersionHeader middleware to

issue: #3140

* docs(relayproxy): add disableVersionHeader to documentation

* feat(relayproxy): fix linter with non used parameter in Skipper

* nit pick to be more idiomatic

Signed-off-by: Thomas Poignant <[email protected]>

---------

Signed-off-by: Thomas Poignant <[email protected]>
Co-authored-by: Tom Flenner <[email protected]>
Co-authored-by: Thomas Poignant <[email protected]>
@kodiakhq kodiakhq bot closed this as completed in #3141 Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📍 Assigned enhancement New feature or request p2 Medium priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants