Skip to content
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

fromEncryptedWallet Errors if PW Longer then 64 bytes #223

Closed
NoahMarconi opened this issue Jul 4, 2018 · 2 comments
Closed

fromEncryptedWallet Errors if PW Longer then 64 bytes #223

NoahMarconi opened this issue Jul 4, 2018 · 2 comments
Assignees
Labels
bug Verified to be an issue. fixed/complete This Bug is fixed or Enhancement is complete and published.

Comments

@NoahMarconi
Copy link

It's an issue with the dependent libraries, you're using a typed array and they expect an array with push.

Locally I fixed by editing this line

scrypt(password, salt, N, r, p, 64, function(error, progress, key) {

to:

                scrypt(Array.from(password), salt, N, r, p, 64, function(error, progress, key) {

I see these are both your repos so not sure if you want the fix here or in the scrypt-js repo. Here's where the error occurs if you pass in a typed array:

https://github.com/ricmoo/scrypt-js/blob/master/scrypt.js#L90

@ricmoo
Copy link
Member

ricmoo commented Jul 5, 2018

Ah yes. I have an open issue in scrypt-js I believe for this too. I will fix it there. I also want to move to using promises in that library, so I’ll update that too.

Thanks! I should be able to get to this very soon.

@ricmoo ricmoo self-assigned this Jul 5, 2018
@ricmoo ricmoo added bug Verified to be an issue. on-deck This Enhancement or Bug is currently being worked on. labels Jul 5, 2018
ricmoo added a commit that referenced this issue Oct 4, 2018
@ricmoo
Copy link
Member

ricmoo commented Oct 4, 2018

This has been fixed in the dependency and the latest version has been updated in the package.json and dist files.

Let me know if there are any problems.

Thanks! :)

@ricmoo ricmoo closed this as completed Oct 4, 2018
@ricmoo ricmoo removed the on-deck This Enhancement or Bug is currently being worked on. label Oct 4, 2018
asnov added a commit to asnov/ethers.js that referenced this issue Oct 14, 2018
Merge commit '3736a1571480a0f69d632d6fc3bde549cbe46162' into fix/tslib

* commit '3736a1571480a0f69d632d6fc3bde549cbe46162': (41 commits)
  Updated dist files.
  Added automatic event parsing for contract transaction receipts from tx.wait.
  Added ability to wait for a specific number of confirmations (ethers-io#229).
  Fix for geth-etc (official geth is fine), which returns Receipts before the blockHash is synced to the database.
  Fixed confirmations tests and bootstrap fast blockNumber.
  Added confirmations to TransactionResponse (ethers-io#156, ethers-io#238).
  Fixed nested errors for providers that were masking true error (ethers-io#292).
  Updated dist files.
  Added version to errors.
  Fixed French and Spanish for browsers without Uint8Array.forEach.
  Added French and Spanish includes to phantomjs test page.
  Increased timeout for querying npm registry.
  Updated dist files.
  Added French and Spanish wordlist dist files.
  Added French and Spanish BIP-39 wordlists (ethers-io#191).
  Added support for JSON serialized BigNumbers in the constructor (ethers-io#288).
  Fixed scrypt for long passwords (ethers-io#223).
  Updated dist files.
  Added chainId as supported override for contract transactions.
  Fixed wildcard events and made nested events more robust (ethers-io#289).
  ...

Conflicts:
	dist/ethers.min.js
	dist/ethers.min.js.map
	package-lock.json
@ricmoo ricmoo added the fixed/complete This Bug is fixed or Enhancement is complete and published. label Jan 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Verified to be an issue. fixed/complete This Bug is fixed or Enhancement is complete and published.
Projects
None yet
Development

No branches or pull requests

2 participants