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.

Sign up for GitHub

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

Jump to bottom

A better implementation for additional scopes #66

Closed
nasatome opened this issue Apr 24, 2019 · 1 comment
Closed

A better implementation for additional scopes #66

nasatome opened this issue Apr 24, 2019 · 1 comment

Comments

@nasatome
Copy link
Contributor

For example, I need to add a scope of Google Drive to not implement a double login with Laravel Socialite (which is the current solution I have).

How can I add scopes: "https://www.googleapis.com/auth/drive" ?
without me returning null? the code Line 134 "return array_get( $scopes, $scope );"

#43
As it is not in the documentation: I can not use this function properly.
nasatome added a commit to nasatome/laravel-gmail that referenced this issue Apr 24, 2019
@nasatome
[fix] Now Accept additional scopes in URL Style dacastro4#66
17cd021 
https://developers.google.com/identity/protocols/googlescopes
@nasatome
Copy link
Contributor Author

fad99c8

Please, check these commits :)

I hope they help the community, as they are helping me.

nasatome added a commit to nasatome/laravel-gmail that referenced this issue Apr 30, 2019
@nasatome
[fix] dacastro4#66 dacastro4#68 correct name in file name when multip…
90f0d60 
…le users are used
dacastro4 pushed a commit that referenced this issue Apr 30, 2019
@nasatome @dacastro4
Enable encryption in json files and accept additional scopes in URL S…
7386e15 
…tyle (#65)

* Enable encryption in json files

* [fix] read non encrypted json

* [fix] Now Accept additional scopes in URL Style #66
https://developers.google.com/identity/protocols/googlescopes

* [chore] indent

* [chore] Update Docs

* [chore] Update Docs for Additional Scopes
dacastro4 pushed a commit that referenced this issue Apr 30, 2019
@nasatome @dacastro4
Fix concatenation of ids (#69)
dd4027d 
* [fix] #66 #68 correct name in file name when multiple users are used
dacastro4 pushed a commit that referenced this issue May 13, 2019
@nasatome @dacastro4
[fix] #71 refresh automatically, If my token expired, in multiUsers (#72
5c4316f 
)

* Enable encryption in json files

* [fix] read non encrypted json

* [fix] Now Accept additional scopes in URL Style #66
https://developers.google.com/identity/protocols/googlescopes

* [chore] indent

* [chore] Update Docs

* [chore] Update Docs for Additional Scopes

* [fix] #71 Refresh token if needed in constructor

* [fix] #71 Save new refreshed Access Token

line#158 Although it is only a line,
there is already a function that performs that task (SOLID)

* [fix] #71 avoid making multiple requests for the refreshed token

* [fix] #71 object to associative array in $savedConfigToken

* [feature] #71 add checkPreviouslyLoggedIn function

* [fix] #71 save null email and return token in refreshTokenIfNeeded

* [fix] #71 checkPreviouslyLoggedIn in constructorfor refresh

* [feature] #71 add email in return makeToken
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dacastro4 @nasatome