Skip to content

Commit

Permalink
Merge pull request #138 from avenirer/patch-1
Browse files Browse the repository at this point in the history
Shouldn't we use urlLogin instead of hardcoding?
  • Loading branch information
hajekj authored Feb 17, 2021
2 parents f2c3c40 + b2fca40 commit 3f8d322
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/Provider/Azure.php
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ protected function getOpenIdConfiguration($tenant, $version) {
}
if (!array_key_exists($version, $this->openIdConfiguration[$tenant])) {
$versionInfix = $this->getVersionUriInfix($version);
$openIdConfigurationUri = 'https://login.microsoftonline.com/' . $tenant . $versionInfix . '/.well-known/openid-configuration';
$openIdConfigurationUri = $this->urlLogin . $tenant . $versionInfix . '/.well-known/openid-configuration';
$factory = $this->getRequestFactory();
$request = $factory->getRequestWithOptions(
'get',
Expand Down

1 comment on commit 3f8d322

@titanlab
Copy link

Choose a reason for hiding this comment

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

Just FYI: If you are working with a Azure B2C with a Custom Policy (IEF) the OpenID Configuration URL is:

https://[tenant].b2clogin.com/[tenant].onmicrosoft.com/[custom-policy]/v2.0/.well-known/openid-configuration

Please sign in to comment.