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

docs: fix okta oauth example for airflow 2.3.3+ #803

Merged
merged 1 commit into from
Nov 15, 2023

Conversation

thesuperzapper
Copy link
Member

@thesuperzapper thesuperzapper commented Nov 15, 2023

What issues does your PR fix?

What does your PR do?

Updates the docs for Okta OAUTH integration to use the format required in Airflow 2.3.3+:

Specifically, you now need to either set server_metadata_url or jwks_uri for OIDC to work.

Checklist

For all Pull Requests

Signed-off-by: Mathew Wicks <5735406+thesuperzapper@users.noreply.github.com>
@thesuperzapper thesuperzapper merged commit 58d96ec into airflow-helm:main Nov 15, 2023
3 checks passed
@thesuperzapper thesuperzapper deleted the update-oauth-docs branch November 15, 2023 21:45
cherusk pushed a commit to cherusk/airflow-helm-charts that referenced this pull request Jan 6, 2024
Signed-off-by: Mathew Wicks <5735406+thesuperzapper@users.noreply.github.com>
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.

None yet

1 participant