csrf-express-angular icon indicating copy to clipboard operation
csrf-express-angular copied to clipboard

Express CSRF syntax change

Open felipe-pereira opened this issue 11 years ago • 4 comments

I followed the steps on your post but found an invalid csrf token error, searched a bit and got the answer here.

As an improvement of that already great post maybe you could include some reference to that change.

Thanks for your work!.

felipe-pereira avatar Jan 14 '15 11:01 felipe-pereira

Would you like to write a pull request?

zemirco avatar Jul 23 '15 13:07 zemirco

Not sure if I could do that since my comment was about updating your post (the one linked in the readme), english is not my native lang so maybe this part wasn't clear enough (well, remove the maybe, I had to read my post a few times to remember what it was about)

felipe-pereira avatar Jul 23 '15 13:07 felipe-pereira

I meant a pull request for this repo updating the code inside https://github.com/zemirco/csrf-express-angular/blob/master/app.js.

zemirco avatar Jul 23 '15 13:07 zemirco

Got it, I could, but at this moment I'm bit short on time and resources to test it properly, the proposed change would require not only a modification to the file you just linked, but also an express update, so it wouldn't be nice of me to pull request that without testing. I would just update the blog post to mention that slight change for newer versions, this app here still would work based on that express version.

felipe-pereira avatar Jul 23 '15 13:07 felipe-pereira