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

docs: Add CSRF protection section #1075

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

masterT
Copy link

@masterT masterT commented Jun 10, 2022

Add a generic section about CSRF protection, to describe how it's implemented in OmniAuth and how to configure it.

References:

@masterT
Copy link
Author

masterT commented Jun 18, 2022

This took me so much time to figure out the CSRF mechanism, I figured I might not be the only one so it could help this section.

@BobbyMcWho
Copy link
Member

There are some grammar changes and misspellings that I need to resolve before merge, I will get to them when I have some time

@masterT
Copy link
Author

masterT commented Jun 24, 2022

@BobbyMcWho I made some changes. Feel free to edit the wording and fix the grammar/spelling mistake, English is not my native language.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@olleolleolle
Copy link

Thanks for including this in the README documentation, it makes it more approachable!

masterT and others added 2 commits September 27, 2022 16:31
Co-authored-by: Olle Jonsson <olle.jonsson@gmail.com>
Co-authored-by: Olle Jonsson <olle.jonsson@gmail.com>
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 this pull request may close these issues.

None yet

3 participants