Skip to content

Use async versions of CertificateCredential and ClientSecretCredential in KiotaRequestAdapterHook#68375

Open
dabla wants to merge 3 commits into
apache:mainfrom
dabla:fix/msgraph-cache-multiplex-asyncio
Open

Use async versions of CertificateCredential and ClientSecretCredential in KiotaRequestAdapterHook#68375
dabla wants to merge 3 commits into
apache:mainfrom
dabla:fix/msgraph-cache-multiplex-asyncio

Conversation

@dabla

@dabla dabla commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Use async versions of CertificateCredential and ClientSecretCredential in KiotaRequestAdapterHook of MSGraph provider, as sync versions could block the event loop and KiotaRequestAdapterHook is a native async hook.


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 force-pushed the fix/msgraph-cache-multiplex-asyncio branch 4 times, most recently from 9a484ff to 076bdbe Compare June 11, 2026 17:39
@dabla dabla requested review from ephraimbuddy, jedcunningham and sunank200 and removed request for ephraimbuddy, jedcunningham and sunank200 June 12, 2026 09:55
…tCredential to avoid blocking the event loop, especially when used concurrently
@dabla dabla force-pushed the fix/msgraph-cache-multiplex-asyncio branch from 076bdbe to cf4b91b Compare June 12, 2026 14:04
@dabla dabla requested review from ephraimbuddy and jedcunningham and removed request for ephraimbuddy and sunank200 June 13, 2026 18:32
@dabla dabla force-pushed the fix/msgraph-cache-multiplex-asyncio branch from ad5ba2e to 1a1c988 Compare June 15, 2026 06:15
@dabla dabla self-assigned this Jun 15, 2026
@dabla dabla force-pushed the fix/msgraph-cache-multiplex-asyncio branch from 9a4442a to 7123edd Compare June 17, 2026 09:11
@dabla dabla force-pushed the fix/msgraph-cache-multiplex-asyncio branch from 7123edd to 0d92d64 Compare June 17, 2026 15:19
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.

1 participant