Skip to content

jQuery Django CRSF helper should be CSRF.#8

Merged
noirbizarre merged 1 commit intonoirbizarre:masterfrom
rouge8:crsf-to-csrf
Feb 22, 2013
Merged

jQuery Django CRSF helper should be CSRF.#8
noirbizarre merged 1 commit intonoirbizarre:masterfrom
rouge8:crsf-to-csrf

Conversation

@rouge8
Copy link
Contributor

@rouge8 rouge8 commented Feb 22, 2013

See the docs.
It stands for Cross-Site Request Forgery.

Breaks backwards compatibility!

It's possible to keep both versions and write tests for each, but you were already using csrf/CSRF in some places and crsf/CRSF in others, so I made it all consistent.

See the [docs](https://docs.djangoproject.com/en/dev/ref/contrib/csrf/).
It stands for Cross-Site Request Forgery.

**Breaks backwards compatibility!**
@cpdean
Copy link

cpdean commented Feb 22, 2013

Travis would be proud.

noirbizarre added a commit that referenced this pull request Feb 22, 2013
jQuery Django `CRSF` helper should be `CSRF`.
@noirbizarre noirbizarre merged commit df2c54f into noirbizarre:master Feb 22, 2013
@noirbizarre
Copy link
Owner

I always misspell this term !

Thanks for the correction !

@rouge8 rouge8 deleted the crsf-to-csrf branch February 23, 2013 04:12
@rouge8 rouge8 restored the crsf-to-csrf branch February 23, 2013 04:13
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.

3 participants