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.

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

Validate webhook using signature instead of re-fetching the event #523

Closed
paltman opened this issue Nov 25, 2017 · 6 comments · Fixed by #620
Closed

Validate webhook using signature instead of re-fetching the event #523

paltman opened this issue Nov 25, 2017 · 6 comments · Fixed by #620
Milestone

Comments

@paltman
Copy link
Contributor

paltman commented Nov 25, 2017

Issue Summary

Stripe now provides webhook signatures so we can drop the old method of protecting against fake events posting to our endpoint by validating the event using the signature.

tysonclugg added a commit to tysonclugg/pinax-stripe that referenced this issue May 16, 2018
@tstirrat15
Copy link

@tysonclugg what's the status of your branch? Will it be going to PR soon? As an interested user, I'd love to see this in a release.

@tysonclugg
Copy link

tysonclugg commented Jun 11, 2018

@tstirrat15 I had some questions regarding updating the docs, which are partially answered. Aside from the docs, my branch is ready for review.
I've been snowed under at work and this got sidelined, but I may have some time today to finish the documentation updates and submit my branch for review.

@tstirrat15
Copy link

Cool. Let me know if I can help in any way.

@tstirrat15
Copy link

@tysonclugg any updates?

@tysonclugg
Copy link

@tstirrat15 I'll have a look on Monday.

@paltman paltman added this to the Light milestone Nov 27, 2021
@paltman
Copy link
Contributor Author

paltman commented Nov 27, 2021

Done in 0d6da79

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

Successfully merging a pull request may close this issue.

3 participants