Dark Mode

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix Keycloak provider redirect_uri to use HTTPS behind reverse proxy #61095

Merged
vincbeck merged 4 commits intoapache:mainfrom
arjav1528:dev-issue-60922
Jan 27, 2026
Merged

Fix Keycloak provider redirect_uri to use HTTPS behind reverse proxy #61095
vincbeck merged 4 commits intoapache:mainfrom
arjav1528:dev-issue-60922

Conversation

Copy link
Contributor

arjav1528 commented Jan 26, 2026 *
edited
Loading

Fixes #60922

The Keycloak authentication provider generates HTTP redirect URLs even when
running behind an HTTPS reverse proxy. This occurs because the login route
uses request.url_for() to generate the callback URL, which doesn't respect
proxy headers like X-Forwarded-Proto by default.

This fix configures Airflow to respect proxy headers by adding support for:

  1. Uvicorn's --proxy-headers flag with FORWARDED_ALLOW_IPS environment variable
  2. Alternative ProxyFix middleware configuration

Both approaches enable the Keycloak provider to correctly generate HTTPS
redirect URLs when deployed behind nginx ingress or other reverse proxies.

Testing:

  • Verified redirect_uri uses HTTPS when proxy headers are configured
  • Confirmed backward compatibility with existing deployments

arjav1528 requested review from hussein-awala, jedcunningham and jscheffl as code owners January 26, 2026 20:37
boring-cyborg bot added area:helm-chart Airflow Helm Chart kind:documentation labels Jan 26, 2026
eladkal requested a review from vincbeck January 26, 2026 21:26
vincbeck approved these changes Jan 26, 2026
vincbeck merged commit 0708036 into apache:main Jan 27, 2026
95 checks passed
shreyas-dev pushed a commit to shreyas-dev/airflow that referenced this pull request Jan 29, 2026
...pache#61095)

* docs: Enhance Airflow API server configuration in values.yaml

* docs: Update Airflow API server args description to include reverse proxy support and provide usage example

* docs: Update API server env vars description to include reverse proxy configuration and provide example usage

* docs: Add Helm chart configuration details for running Airflow behind a reverse proxy
jedcunningham mentioned this pull request Jan 30, 2026
98 tasks
shashbha14 pushed a commit to shashbha14/airflow that referenced this pull request Feb 2, 2026
...pache#61095)

* docs: Enhance Airflow API server configuration in values.yaml

* docs: Update Airflow API server args description to include reverse proxy support and provide usage example

* docs: Update API server env vars description to include reverse proxy configuration and provide example usage

* docs: Add Helm chart configuration details for running Airflow behind a reverse proxy
jason810496 pushed a commit to abhijeets25012-tech/airflow that referenced this pull request Feb 3, 2026
...pache#61095)

* docs: Enhance Airflow API server configuration in values.yaml

* docs: Update Airflow API server args description to include reverse proxy support and provide usage example

* docs: Update API server env vars description to include reverse proxy configuration and provide example usage

* docs: Add Helm chart configuration details for running Airflow behind a reverse proxy
jhgoebbert pushed a commit to jhgoebbert/airflow_Owen-CH-Leung that referenced this pull request Feb 8, 2026
...pache#61095)

* docs: Enhance Airflow API server configuration in values.yaml

* docs: Update Airflow API server args description to include reverse proxy support and provide usage example

* docs: Update API server env vars description to include reverse proxy configuration and provide example usage

* docs: Add Helm chart configuration details for running Airflow behind a reverse proxy
choo121600 pushed a commit to choo121600/airflow that referenced this pull request Feb 22, 2026
...pache#61095)

* docs: Enhance Airflow API server configuration in values.yaml

* docs: Update Airflow API server args description to include reverse proxy support and provide usage example

* docs: Update API server env vars description to include reverse proxy configuration and provide example usage

* docs: Add Helm chart configuration details for running Airflow behind a reverse proxy
Subham-KRLX pushed a commit to Subham-KRLX/airflow that referenced this pull request Mar 4, 2026
...pache#61095)

* docs: Enhance Airflow API server configuration in values.yaml

* docs: Update Airflow API server args description to include reverse proxy support and provide usage example

* docs: Update API server env vars description to include reverse proxy configuration and provide example usage

* docs: Add Helm chart configuration details for running Airflow behind a reverse proxy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

vincbeck vincbeck approved these changes

jedcunningham Awaiting requested review from jedcunningham jedcunningham is a code owner

hussein-awala Awaiting requested review from hussein-awala hussein-awala is a code owner

jscheffl Awaiting requested review from jscheffl jscheffl is a code owner

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Airflow Keycloak provider redirect to HTTP instead of HTTPS

2 participants