Skip to content

Conversation

fidoriel
Copy link

Currently, only x-forwarded-* headers are used for original host extraction in decision api.
Nginx defaults to x-origin-* deployed as ingress. This PR does also check the default nginx headers.

Locally, tests are failing because I dont have azure credentials.

Related issue(s)

closes #521

Checklist

  • I have read the contributing guidelines.
  • I have referenced an issue containing the design document if my change
    introduces a new feature.
  • I am following the
    contributing code guidelines.
  • I have read the security policy.
  • I confirm that this pull request does not address a security
    vulnerability. If this pull request addresses a security vulnerability, I
    confirm that I got the approval (please contact
    [email protected]) from the maintainers to push
    the changes.
  • I have added tests that prove my fix is effective or that my feature
    works.
  • I have added or changed the documentation.

@fidoriel fidoriel requested review from a team and aeneasr as code owners March 20, 2025 14:35
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Decision API support for the nginx-ingress annotation nginx.ingress.kubernetes.io/auth-url

2 participants