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 DAJAXICE_FETCH_CSRF_FROM_FIELD in order to avoid reading CSRF cookie #109

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

Conversation

raymondr
Copy link

A security recommendation is to mark all cookies httponly which disallows javascript from reading the cookies. This causes problems with dajaxice. This change adds a configuration option to pull the CSRF token from the form field rather than the cookie.

raymondr added 2 commits June 26, 2013 13:29
Implement DAJAXICE_FETCH_CSRF_FROM_FIELD
If you have set httponly attribute on the CSRFcookie, javascript will not be able to read CSRF cookie. Instead we can fetch the CSRF token from the hidden field generated in the form by CSRF middleware.
@maldn
Copy link

maldn commented Jun 26, 2013

Hallo,
Herr Böhme befindet sich bis zum 02.05. im Urlaub.
Bei technischen Fragen wenden Sie sich bitte an Herrn Eike Foken ([email protected]).

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.

2 participants