Skip to content
This repository has been archived by the owner on Feb 25, 2019. It is now read-only.

AES-CTR encrypt/decrypt: implement missing Crypto support #35

Closed
thelunararmy opened this issue Jun 29, 2017 · 2 comments
Closed

AES-CTR encrypt/decrypt: implement missing Crypto support #35

thelunararmy opened this issue Jun 29, 2017 · 2 comments
Labels
low priority needs investigating
Milestone
v1.0.0

Comments

@thelunararmy
Copy link
Contributor

thelunararmy commented Jun 29, 2017 •
edited
Loading

As outlined in https://www.w3.org/TR/WebCryptoAPI/#dfn-AesCtrParams : which requires the use of a counter and length parameters for use in the counter style block encryption (More info here), is currently not "intuitively" supported by the node's lib Crypto https://nodejs.org/api/crypto.html#crypto_class_cipher. It has some base level implementation of aes-128-ctr, but does not make use of a counter at all; thus resulting in an unsupported issue complained about in nodejs/node#13801.

@EternalDeiwos pointed out https://github.com/jrnewell/crypto-aes-ctr/blob/master/lib/index.js as good alternative. Further investigation may need to be done if node doesnt support this problem.
@thelunararmy thelunararmy added this to the v1.0.0 milestone Jun 29, 2017
@christiansmith
Copy link
Member

This would be useful for implementing Ethereum/Web3 Secret Storage Definition.

https://github.com/ethereum/wiki/wiki/Web3-Secret-Storage-Definition

@thelunararmy
Copy link
Contributor Author

Fixed, now supported as of 0.7.0

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
low priority needs investigating
Projects
None yet
Milestone
v1.0.0
Development

No branches or pull requests
2 participants
@christiansmith @thelunararmy