-
-
Notifications
You must be signed in to change notification settings - Fork 18k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # 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