Bugs - Azure B2C - URL char issue

[massej]

Hi

This part has a bug

   public function requestUserInfo($attribute = null) {

        $user_info_endpoint = $this->getProviderConfigValue('userinfo_endpoint');
        $schema = 'openid';

        $user_info_endpoint .= '?schema=' . $schema;

        //The accessToken has to be sent in the Authorization header.
        // Accept json to indicate response type
        $headers = ["Authorization: Bearer {$this->accessToken}",
            'Accept: application/json'];

The Azure B2C use this URL for the user endpoint

https://tenant.b2clogin.com/tenant.onmicrosoft.com/openid/v2.0/userinfo?p=b2c_1a_demo_susi_username

But the line
$user_info_endpoint .= '?schema=' . $schema;

should be with & instead.

It cause the problem that it cause a 404 due to trying to get

https://tenant.b2clogin.com/tenant.onmicrosoft.com/openid/v2.0/userinfo?p=b2c_1a_demo_susi_username?schema=openid instead of doing this

https://tenant.b2clogin.com/tenant.onmicrosoft.com/openid/v2.0/userinfo?p=b2c_1a_demo_susi_username&schema=openid

So it there is no ? inside the URL then it should add & instead and it will fix the issue,

Thank you for your help,

[ricklambrechts]

We could check if there is a ? In the url. If so use &. Thanks!

Do you want to make a PR for this? I can do it also.

[massej]

Thank you @ricklambrechts I just open a pull request