Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Will there be support on verifying Facebook Graph API Calls with appsecret_proof? #1165

Closed
XY-Wang opened this issue Dec 7, 2017 · 2 comments

Comments

@XY-Wang
Copy link

XY-Wang commented Dec 7, 2017

Hi guys,

I've noticed recently that there has been very suspicious behaviour on one of our bots where there are lots of concurrent new user sessions being created which do not appear as conversations in the bot FB page. This completely overloads the bot and normal user messages can't get through. I applied validate_requests: true to my bot and it seems to hold off this behaviour. So, many thanks for developing this and shame on me for not applying it sooner.

On another note, I've been reading about FB security and came across the "Require app secret" setting in the app page:
screen shot 2017-12-07 at 1 28 58 pm

We can secure our API request by adding a parameter appsecret_proof as described in FB's documentations: https://developers.facebook.com/docs/graph-api/securing-requests/

Will Botkit support this in the near future? It would be great to have additional security measures to avoid other possible suspicious behaviour.

@XY-Wang XY-Wang changed the title Will there be support on verifying FB Graph API Calls with appsecret_proof? Will there be support on verifying Facebook Graph API Calls with appsecret_proof? Dec 7, 2017
@ouadie-lahdioui
Copy link
Collaborator

ouadie-lahdioui commented Dec 8, 2017

Yes, Botkit will support this FB recommandation, PR is already here #1170.

Thanks for the report ;)

@peterswimm
Copy link
Contributor

Closing this for now, please follow the above linked thread for progress!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants