Skip to content

Fix single-tenant authentication redirect URI handling and add administrator consent support - #3745

Merged
abraunegg merged 1 commit into
masterfrom
update-display-consent-url
Jun 9, 2026
Merged

abraunegg merged 1 commit into
masterfrom
update-display-consent-url

Conversation

@abraunegg

Copy link
Copy Markdown
Owner

Fix authentication URL generation when azure_tenant_id is configured and the default OneDrive Client for Linux application registration is being used.

Changes include:

  • Fix redirect URI generation for single-tenant authentication when using the default application registration
  • Retain the registered Microsoft Identity Platform native-client redirect URI: https://login.microsoftonline.com/common/oauth2/nativeclient
  • Update the --display-admin-consent-url CLI option
  • Generate tenant-specific Microsoft Entra ID administrator consent URLs
  • Improve documentation for azure_tenant_id
  • Add documentation covering Microsoft Entra ID administrator consent workflows and tenant-wide application approval

These changes resolve Microsoft Entra ID authentication failures reporting AADSTS50011 when using azure_tenant_id with the default application registration and provide a documented workflow for administrator consent in managed Microsoft 365 and Microsoft Entra ID environments.

…strator consent support

Fix authentication URL generation when `azure_tenant_id` is configured and the default OneDrive Client for Linux application registration is being used.

Changes include:

* Fix redirect URI generation for single-tenant authentication when using the default application registration
* Retain the registered Microsoft Identity Platform native-client redirect URI:
  `https://login.microsoftonline.com/common/oauth2/nativeclient`
* Add the `--display-admin-consent-url` CLI option
* Generate tenant-specific Microsoft Entra ID administrator consent URLs
* Improve documentation for `azure_tenant_id`
* Add documentation covering Microsoft Entra ID administrator consent workflows and tenant-wide application approval

These changes resolve Microsoft Entra ID authentication failures reporting `AADSTS50011` when using `azure_tenant_id` with the default application registration and provide a documented workflow for administrator consent in managed Microsoft 365 and Microsoft Entra ID environments.
@abraunegg abraunegg added this to the v2.5.11 milestone Jun 9, 2026
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Business Account Shared Folder Testing

5 Test Cases Run
5 Test Cases Passed
0 Test Cases Failed

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Personal Account Shared Folder Testing

3 Test Cases Run
3 Test Cases Passed
0 Test Cases Failed

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Sharepoint Account Testing

64 Test Cases Run
64 Test Cases Passed
0 Test Cases Failed

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Business Account Testing

64 Test Cases Run
64 Test Cases Passed
0 Test Cases Failed

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Personal Account Testing - 15 Character 'driveId'

64 Test Cases Run
64 Test Cases Passed
0 Test Cases Failed

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Personal Account Testing

1 Test Cases Run
1 Test Cases Passed
0 Test Cases Failed

⚠️ Primary run had transient failure(s), but all failed case(s) passed during debug rerun. Treating workflow as passed.

@abraunegg
abraunegg merged commit 7793830 into master Jun 9, 2026
14 checks passed
@abraunegg
abraunegg deleted the update-display-consent-url branch June 10, 2026 20:53
@abraunegg

Copy link
Copy Markdown
Owner Author

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

Repository owner locked and limited conversation to collaborators Jun 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant