Skip to content

Commit

Permalink
Merge pull request #15 from StuBez/scope-separator
Browse files Browse the repository at this point in the history
Specify scope separator in Oauth provider
  • Loading branch information
hailwood authored Jun 8, 2020
2 parents 3c0c366 + 40633ad commit 45b5cef
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
7 changes: 5 additions & 2 deletions src/Oauth2Provider.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,8 @@

class Oauth2Provider extends GenericProvider
{

}
protected function getScopeSeparator()
{
return ' ';
}
}
2 changes: 1 addition & 1 deletion src/OauthCredentialManager.php
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ public function getState()

public function getAuthorizationUrl()
{
$redirectUrl = $this->oauthProvider->getAuthorizationUrl(['scope' => implode(' ', config('xero.oauth.scopes'))]);
$redirectUrl = $this->oauthProvider->getAuthorizationUrl(['scope' => config('xero.oauth.scopes')]);
$this->session->put($this->cacheKey, $this->oauthProvider->getState());

return $redirectUrl;
Expand Down

0 comments on commit 45b5cef

Please sign in to comment.