Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid example in README #217

Open
r3h6 opened this issue Jan 21, 2025 · 1 comment
Open

Invalid example in README #217

r3h6 opened this issue Jan 21, 2025 · 1 comment

Comments

@r3h6
Copy link

r3h6 commented Jan 21, 2025

In the example code of the README the scope property is defined as string, but it must be an array.

Invalid code: $provider->scope = 'openid profile email offline_access ' . $baseGraphUri . '/User.Read';

@decomplexity
Copy link

From memory, 'scopes' can be specified as either a string of space-separated substrings or as an array of strings.
AbstractProvider.php tests if the scope specified is an array; if so, it implodes the array into a string of substrings that are delimited by scopeSeparator.
TheNetworg's Azure.php specifies public property scopeSeparator as ' ' (i.e. a single space)

[I am currently travelling so perhaps OP can try both and report back Tnx]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants