search

expressjs / csurf

CSRF token middleware
MIT License
2.3k stars 216 forks source link

Update split token generation/validation branch from master #91

Open strawbrary opened 8 years ago

strawbrary commented 8 years ago

Merged the latest changes from the master branch, resolved the conflicts, fixed some styles that were inconsistent with the existing code and added some simple documentation for the new API to the README.

twelve17 commented 7 years ago

Any updates on this?

strawbrary commented 7 years ago

@twelve17 My guess is @dougwilson still has a long backlog of PRs to review given how many projects he manages. If you want to try it out, I've published the fork temporarily on npm under the name csurfer.

dougwilson commented 7 years ago

I do, and I got an opportunity to start plowing through them as of a week ago, so will get getting here after not too long. Really, the split-module branch just needs to be rebased, as merges are not part of this org's workflow anyway :)

strawbrary commented 7 years ago

Oh ok I wasn't aware of that. I'll try to submit a new PR soon following the correct protocol.

dougwilson commented 7 years ago

It's no problem. I'm not sure if you actually can do that work via PR at all.