-
-
Notifications
You must be signed in to change notification settings - Fork 16.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade cookie signature #5833
Upgrade cookie signature #5833
Conversation
@wesleytodd do you want me to rebase it to update with latest pushes in the master? Edit: already did 🙂 |
e6d3a58
to
e52abcd
Compare
package.json
Outdated
@@ -33,7 +33,7 @@ | |||
"content-disposition": "0.5.4", | |||
"content-type": "~1.0.4", | |||
"cookie": "0.6.0", | |||
"cookie-signature": "1.0.6", | |||
"cookie-signature": "1.2.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we open it up with ^
? We ahve been doing that as we update these in the v5 branch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for pointing out the caret.
In accordance with the expressjs/discussions#256, we are upgrading the
cookie-signature
from 1.0.6 to 1.2.1.Notes
Since all items listed in list are already landed, this PR