Skip to content

feat: add disable-forwarded-header to NIC ConfigMap - #2173

Draft
antojacob wants to merge 2 commits into
nic-release-5.6from
nic/use-forwarded-header
Draft

feat: add disable-forwarded-header to NIC ConfigMap #2173
antojacob wants to merge 2 commits into
nic-release-5.6from
nic/use-forwarded-header

Conversation

@antojacob

@antojacob antojacob commented Jul 27, 2026

Copy link
Copy Markdown

Proposed changes

Implementation PR - nginx/kubernetes-ingress#10539

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

@antojacob
antojacob requested a review from a team as a code owner July 27, 2026 14:57
@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown

✅ All required contributors have signed the F5 CLA for this PR. Thank you!
Posted by the CLA Assistant Lite bot.

@github-actions github-actions Bot added documentation Improvements or additions to documentation product/nic Issues related to NGINX Ingress Controller labels Jul 27, 2026
@github-actions

Copy link
Copy Markdown

Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/docs/2173/

@AlexFenlon AlexFenlon assigned AlexFenlon and unassigned AlexFenlon Jul 27, 2026
@AlexFenlon
AlexFenlon marked this pull request as draft July 27, 2026 16:28
Comment thread content/nic/configuration/global-configuration/configmap-resource.md Outdated
…rce.md

Co-authored-by: yar <y82@users.noreply.github.com>

@medikondasoniya medikondasoniya left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@y82

y82 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Hi @antojacob, thanks for the contribution!
In order for the PR to proceed, you need to accept the the F5 Contributor License Agreement (CLA), by leaving the following message here:

"I have hereby read the F5 CLA and agree to its terms"

Thanks!

@AlexFenlon AlexFenlon changed the title feat: add use-forwarded-header to NIC ConfigMap feat: add use-forwarded-header to NIC ConfigMap [ON HOLD] Jul 29, 2026
@AlexFenlon

AlexFenlon commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Implementation PR on hold for now, Anto will accept F5 CLA soon but lets not merge this until we are ready

@AlexFenlon
AlexFenlon marked this pull request as draft July 29, 2026 08:44
@antojacob antojacob changed the title feat: add use-forwarded-header to NIC ConfigMap [ON HOLD] feat: add disable-forwarded-header to NIC ConfigMap Jul 29, 2026
@antojacob

Copy link
Copy Markdown
Author

I have hereby read the F5 CLA and agree to its terms

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

Labels

documentation Improvements or additions to documentation product/nic Issues related to NGINX Ingress Controller

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants