-
Notifications
You must be signed in to change notification settings - Fork 147
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
add appsecret_proof support, useful for post 2.8 api version #170
Conversation
Has the change been tested? |
bonus, semicolon is fixed. |
TODO: Update CHANGELOG.md |
Aye aye Sir, well tested :) |
Yeah, that's not a mistake, but it's ugly. |
Done |
add space
I think it would be cleaner and more beautiful to take app_secret from the config and automatically use it for version v2.8. Thus, we would get rid of the unnecessary parameter in the constructor. |
From config? |
env('FACEBOOK_APP_SECRET'); |
sorry, I forgot it wasn't just for laravel. |
add appsecret_proof support, useful for post 2.8 api version
see Securing Graph API Requests - Facebook for Developers