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

Add asgi-csrf #69

Merged
merged 3 commits into from
Jun 3, 2022
Merged

Add asgi-csrf #69

merged 3 commits into from
Jun 3, 2022

Conversation

florimondmanca
Copy link
Owner

@florimondmanca florimondmanca commented Oct 18, 2021

Checklist

  • This project is explicitly related to ASGI.
  • The new list entry contains a project name, URL and description.

What is this project?

asgi-csrf is @simonw's CSRF-for-ASGI implementation. It is widely used in the Datasette ecosystem I believe: https://github.com/simonw/asgi-csrf/network/dependents

https://github.com/simonw/asgi-csrf

Do you know about other similar projects?

Yes - there's the CSRFMiddleware already listed in this list.

If so, how is this one different?

asgi-csrf is standalone, while the CSRFMiddleware we have is part of a wider library (Piccolo API). Makes me think we might want to put asgi-csrf as the lone entry later, maybe.


Anyone who agrees with this pull request can add a 👍.

@florimondmanca florimondmanca added the please vote Vote to confirm interest in adding this project to the list label Oct 18, 2021
@florimondmanca florimondmanca added candidate Project is relevant to include in the list please vote Vote to confirm interest in adding this project to the list and removed please vote Vote to confirm interest in adding this project to the list candidate Project is relevant to include in the list labels Oct 18, 2021
@florimondmanca florimondmanca merged commit fde2e06 into master Jun 3, 2022
@florimondmanca florimondmanca deleted the fm/asgi-csrf branch June 3, 2022 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
please vote Vote to confirm interest in adding this project to the list
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant