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
Line provider #997
base: master
Are you sure you want to change the base?
Line provider #997
Conversation
LGTM |
@@ -16,7 +16,8 @@ | |||
"gitter": "https://gitter.im/hybridauth/hybridauth" | |||
}, | |||
"require": { | |||
"php": ">=5.4.0" | |||
"php": ">=5.4.0", | |||
"firebase/php-jwt": "^5.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not fan of unnecessary dependencies. Ideally JWT::decode would be contained in the adapter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I created nitipatl#1 where JWT
is moved to third-party packages of Hybridauth.
Is this OK. I think it is because JWT
will maybe be also used by other packages in the future.
So, @nitipatl should merge that PR, so it will be added to this.
Do you have any suggest?