Skip to content

Add hints for claim validation to auth manager documentation#59320

Merged
vincbeck merged 1 commit into
apache:mainfrom
boschglobal:feature/extend-documentation-for-custom-api-auth-manager
Dec 11, 2025
Merged

Add hints for claim validation to auth manager documentation#59320
vincbeck merged 1 commit into
apache:mainfrom
boschglobal:feature/extend-documentation-for-custom-api-auth-manager

Conversation

@wolfdn
Copy link
Copy Markdown
Contributor

@wolfdn wolfdn commented Dec 11, 2025

Description

This PR adds some important hints to the documentation that describes how to create a custom auth manager for API authentication. Especially mentioning that the iss (issuer) needs to be validated is important, because otherwise it cannot be guaranteed that JWTs that are sent to the API were issued by a trusted issuer (when working with Entra ID there might be tenants that do not belong to your organization).


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

Copy link
Copy Markdown
Contributor

@vincbeck vincbeck left a comment

Choose a reason for hiding this comment

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

Nice!

Copy link
Copy Markdown
Member

@choo121600 choo121600 left a comment

Choose a reason for hiding this comment

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

GOOD :)

@vincbeck vincbeck merged commit 95adec4 into apache:main Dec 11, 2025
126 of 128 checks passed
@potiuk
Copy link
Copy Markdown
Member

potiuk commented Dec 11, 2025

Nice indeed :)

jhgoebbert pushed a commit to jhgoebbert/airflow_Owen-CH-Leung that referenced this pull request Feb 8, 2026
Subham-KRLX pushed a commit to Subham-KRLX/airflow that referenced this pull request Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants