Skip to content

Invalidate cached RequestAdapter of KiotaRequestAdapterHook if session of AuthenticationProvider is closed#69128

Merged
dabla merged 2 commits into
apache:mainfrom
dabla:fix/msgraph-session-closed
Jul 3, 2026
Merged

Invalidate cached RequestAdapter of KiotaRequestAdapterHook if session of AuthenticationProvider is closed#69128
dabla merged 2 commits into
apache:mainfrom
dabla:fix/msgraph-session-closed

Conversation

@dabla

@dabla dabla commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Check if session of credentials provider in cached RequestAdapter of KiotaRequestAdapterHook isn't closed, if so invalidate the cached RequestAdapter.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@dabla dabla self-assigned this Jun 29, 2026
@dabla dabla changed the title Check if session of credentials provider in cached RequestAdapter of KiotaRequestAdapterHook isn't closed Invalidate cached RequestAdapter of KiotaRequestAdapterHook if session of AuthenticationProvider is closed Jun 29, 2026
@dabla dabla force-pushed the fix/msgraph-session-closed branch 3 times, most recently from a696020 to 6a12ee7 Compare June 30, 2026 12:16

@SameerMesiah97 SameerMesiah97 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Needs one clarification. I have left a few comments.

…uestAdapter isn't closed for KiotaRequestAdapterHook, if so invalidate the cached RequestAdapter
@dabla dabla force-pushed the fix/msgraph-session-closed branch from 6a12ee7 to 8005600 Compare July 1, 2026 16:39
@dabla dabla marked this pull request as draft July 2, 2026 07:06
@dabla dabla marked this pull request as ready for review July 2, 2026 21:44
@dabla dabla merged commit 49f3d7f into apache:main Jul 3, 2026
157 checks passed
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.

3 participants