Saltshaker-Crypto

Security and privacy

encryption library

SHOW HN2019

Saltshaker-Crypto

Saltshaker-Crypto, NPM to Sign, Verify, Encrypt and Decrypt with Nacl

npmjs.com

#nacl#tweetnacl#encryption#javascript#npm

20points
7comments
#1,193of 2,195 in 2019
Top 55%of 2019 by points

How HN took it

RoastedHalf or more of the comments are critical

0 positive, 0 neutral or asking, 0 mixed, 5 critical, out of 5 comments about the project. Thread score -0.50.

Privacy: 0 for, 3 againstExecution: 0 for, 1 againstIdea: 0 for, 1 against

The thread, comment by comment

Each row is a top-level comment and its replies, in HN's order. Select a cell to open that comment on HN.

positive neutral or mixed critical about another commenter or something else the maker

What HN said

What people liked

  • tweetnacl-js is a well-regarded, security-audited library

    From 1 comment
    • Seems like the right library to post is https://github.com/dchest/tweetnacl-js which actually has been security audited
      jitl on HN

Objections

  • Included unauthenticated AES-CBC encryption, which is insecure and has padding oracle risks

    From 3 comments
    • Help me understand. You've written a small Javascript wrapper around 'dchest's Javascript port of tweetnacl, and then added to it the ability to insecurely encrypt and decrypt…
      tptacek on HN
    • It is really strange that they added another library to do unauthenticated AES when nacl has a symmetric AEAD mode...
      jszymborski on HN
    • Wasn’t use of CBC strongly discouraged because it allowed for padding oracles ?
      stingraycharles on HN
  • Wrapping a library adds little value over using tweetnacl directly

    From 1 comment
    • Help me understand. You've written a small Javascript wrapper around 'dchest's Javascript port of tweetnacl, and then added to it the ability to insecurely encrypt and decrypt…
      tptacek on HN

Requests

  • Remove the AES-CBC code and use NaCl's AEAD mode instead

    From 2 comments
    • It is really strange that they added another library to do unauthenticated AES when nacl has a symmetric AEAD mode...
      jszymborski on HN
    • Revisit the AES by removing the AES.
      tptacek on HN

Questions

  • Why add unauthenticated AES alongside NaCl

    From 2 comments
    • Help me understand. You've written a small Javascript wrapper around 'dchest's Javascript port of tweetnacl, and then added to it the ability to insecurely encrypt and decrypt…
      tptacek on HN
    • It is really strange that they added another library to do unauthenticated AES when nacl has a symmetric AEAD mode...
      jszymborski on HN

Alternatives named

  • tweetnacl-js (dchest)

    From 1 comment
    • Seems like the right library to post is https://github.com/dchest/tweetnacl-js which actually has been security audited
      jitl on HN

Category, keywords, summaries and labels on this page were made by AI (Thread Reader v2) and not checked by hand. Each point links to the comments it came from.

Add a badge to your READMEShown on HackerShow
[![Shown on HackerShow](https://hackershow.org/badge.svg)](https://hackershow.org/show/19214333)