Skip to content

Commit

Permalink
Updated AzureAD provider to use microsoft graph
Browse files Browse the repository at this point in the history
Since AzureAD graph is going away.
Tested using old AzureAD graph usage for backwards-compatbility, did not
seem to break things. Could not test with conditional access though due
to azure never enforcing it no matter what I attempted.

Fpr #3028
  • Loading branch information
ssddanbrown committed Oct 31, 2021
1 parent 85dc8d9 commit 5c834f2
Show file tree
Hide file tree
Showing 3 changed files with 75 additions and 68 deletions.
3 changes: 0 additions & 3 deletions app/Auth/Access/SocialAuthService.php
Expand Up @@ -281,9 +281,6 @@ protected function getDriverForRedirect(string $driverName): Provider
if ($driverName === 'google' && config('services.google.select_account')) {
$driver->with(['prompt' => 'select_account']);
}
if ($driverName === 'azure') {
$driver->with(['resource' => 'https://graph.windows.net']);
}

if (isset($this->configureForRedirectCallbacks[$driverName])) {
$this->configureForRedirectCallbacks[$driverName]($driver);
Expand Down
2 changes: 1 addition & 1 deletion composer.json
Expand Up @@ -33,7 +33,7 @@
"predis/predis": "^1.1.6",
"socialiteproviders/discord": "^4.1",
"socialiteproviders/gitlab": "^4.1",
"socialiteproviders/microsoft-azure": "^4.1",
"socialiteproviders/microsoft-azure": "^5.0.1",
"socialiteproviders/okta": "^4.1",
"socialiteproviders/slack": "^4.1",
"socialiteproviders/twitch": "^5.3",
Expand Down

0 comments on commit 5c834f2

Please sign in to comment.