Merge pull request #419 from leansoft/master

Fixed scope delimiter for github
daviddesberg · Sep 17, 2015 · 824e138 · 824e138
2 parents 24bc167 + f5aeb10
commit 824e138
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/src/OAuth/OAuth2/Service/Facebook.php b/src/OAuth/OAuth2/Service/Facebook.php
@@ -197,6 +197,7 @@ public function getDialogUri($dialogPath, array $parameters)
         $query = http_build_query($parameters);
         return new Uri($baseUrl . '?' . $query);
     }
+
     /**
      * {@inheritdoc}
      */

diff --git a/src/OAuth/OAuth2/Service/GitHub.php b/src/OAuth/OAuth2/Service/GitHub.php
@@ -205,4 +205,12 @@ protected function getExtraApiHeaders()
     {
         return array('Accept' => 'application/vnd.github.beta+json');
     }
+
+    /**
+     * {@inheritdoc}
+     */
+    protected function getScopesDelimiter()
+    {
+        return ',';
+    }
 }
-Original file line number
+Diff line change
@@ Expand Up @@
             $query = http_build_query($parameters);
             return new Uri($baseUrl . '?' . $query);
         }
         /**
          * {@inheritdoc}
          */
@@ Expand Down @@