csurf icon indicating copy to clipboard operation
csurf copied to clipboard

Published 20 hours ago • verified publisher icon expressjs

Update split token generation/validation branch from master

Open strawbrary opened this issue 9 years ago • 5 comments

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.

strawbrary avatar Mar 03 '16 08:03 strawbrary

Any updates on this?

twelve17 avatar Nov 08 '16 10:11 twelve17

@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.

strawbrary avatar Nov 13 '16 06:11 strawbrary

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 :)

dougwilson avatar Nov 13 '16 07:11 dougwilson

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

strawbrary avatar Nov 13 '16 07:11 strawbrary

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

dougwilson avatar Nov 13 '16 07:11 dougwilson