Skip to content
This repository has been archived by the owner on Nov 8, 2022. It is now read-only.

Added improved implementation of CSRF guard with ajax protection #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

siberex
Copy link

@siberex siberex commented Aug 19, 2014

Added CSRF guard with request header check, compatible with AngularJS cookie & header names by default.

Added configuration options including action on token check fail to address issues like this: http://help.slimframework.com/discussions/questions/912-catch-csrf-error

Added helpers for Twig to ease template updates and jQuery configuration

@tuupola
Copy link

tuupola commented Aug 20, 2014

Personally I think you are better off creating separate repository and composer package. That way there will be only one middleware in the commit history. People can also choose which middleware they install instead of installing bunch of unrelated middlewares at once.

@siberex
Copy link
Author

siberex commented Aug 20, 2014

This PR intended as a full replacement of CsrfGuard from Slim-Extras for this repo. If I understand correctly, this repo is a replacement for middleware part of deprecated Slim-Extras. Correct me if I am wrong. My intention was to rewrite old CsrfGuard from scratch and fix all of its issues.

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

Successfully merging this pull request may close these issues.

None yet

2 participants