changed the hashing algorithm from sha1 to sha256 due to security problems by lucianidev · Pull Request #990 · expressjs/session · GitHub
Skip to content

changed the hashing algorithm from sha1 to sha256 due to security problems#990

Open
lucianidev wants to merge 1 commit intoexpressjs:masterfrom
lucianidev:master
Open

changed the hashing algorithm from sha1 to sha256 due to security problems#990
lucianidev wants to merge 1 commit intoexpressjs:masterfrom
lucianidev:master

Conversation

@lucianidev
Copy link
Copy Markdown

hello,
in the issue #989 the user references a vulnerability in the hashing algorithm sha1. The hashing method resulted vulnerable to hash collision, making it insecure. I changed the hashing method from sha1 to sha256 making it more secure. Tell me about any problems.
I hope you have a good day

Documentation:
https://crypto.stackexchange.com/questions/48289/how-secure-is-sha1-what-are-the-chances-of-a-real-exploit
https://www.quora.com/How-secure-is-SHA1-What-are-the-chances-of-a-real-exploit
https://stackoverflow.com/questions/38038841/why-is-sha-1-considered-insecure

@jonchurch
Copy link
Copy Markdown
Member

@krko12345
Copy link
Copy Markdown

Hello team, is this going to be addressed at some point soon?
Session is being reported as vulnerability by monitoring tools because of its use of sha1.

@bjohansebas
Copy link
Copy Markdown
Member

One option would be to allow the user to choose which hashing algorithm to use

@jonchurch
Copy link
Copy Markdown
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants