-
Notifications
You must be signed in to change notification settings - Fork 664
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
Create a new @slack/verify-request package #1135
Comments
We should do this. As Bolt for JS has a similar code, we can replace the code in Bolt for JS with this new package in Bolt's next major version. As far as I know, the points to consider are:
References: |
The verify-request does not seem to be working for the event calls, as the request body seems to be coming serialized now.. |
Hi @mim-Armand,
This situation might be caused by some settings in your application. For instance, if your app is built with Express.js, having Express middleware may cause this. If you have more to ask on this, please feel free to create a new issue for the question! (this issue #1135 is for the task to create a new package, so please avoid continuing this conversation here) |
👋 It looks like this issue has been open for 30 days with no activity. We'll mark this as stale for now, and wait 10 days for an update or for further comment before closing this issue out. |
Description
We should rip out the
verifyRequestSignature
code out of@slack/events-api
and make it a stand alone package named@slack/verify-request
.What type of issue is this? (place an
x
in one of the[ ]
)Requirements (place an
x
in each of the[ ]
)The text was updated successfully, but these errors were encountered: