Merge pull request #3 from sdaoudi/sdaoudi-patch-1

Update keycloak scope separator

src/Provider/Keycloak.php

@@ -174,9 +174,20 @@ protected function getBaseUrlWithRealm()
  */
  protected function getDefaultScopes()
  {
- return ['name', 'email'];
+ return ['profile', 'email'];
  }
-
+
+ /**
+ * Returns the string that should be used to separate scopes when building
+ * the URL for requesting an access token.
+ *
+ * @return string Scope separator, defaults to ','
+ */
+ protected function getScopeSeparator()
+ {
+ return ' ';
+ }
+
  /**
  * Check a provider response for errors.
  *

test/src/Provider/KeycloakTest.php

@@ -132,7 +132,7 @@ public function testEncryptionKeyPathFails()
 
  public function testScopes()
  {
- $scopeSeparator = ',';
+ $scopeSeparator = ' ';
  $options = ['scope' => [uniqid(), uniqid()]];
  $query = ['scope' => implode($scopeSeparator, $options['scope'])];
  $url = $this->provider->getAuthorizationUrl($options);