AElfProject / aelf-web3.js

aelf JavaScript SDK
MIT License
16 stars 24 forks source link

fix: 🐛 fix sha256 incorrect in google chrome #114

Closed potter-sun closed 1 year ago

potter-sun commented 1 year ago

Google Chrome 115.0.5790.102 produces incorrect SHA-256 hashes

When js-sha256 is used in chrome, there will be occasional problems in generating hash. Refer to the following method to use @aws-crypto/sha256-js

reference link: js-sha issue

vizipi[bot] commented 1 year ago

Pull request analysis by VIZIPI

Below you will find who is the most qualified team member to review your code. This analysis includes his/her work on the code included in this Pull request, in addition to their experience in code affected by these changes ( partly found within the list of potential missing files below )   Feedback always welcome

Reviewers with knowledge related to these changes

Match % Person Commit Count Common Files
75.00 % AbigailDeng 46 9
33.33 % lhoopox 2 4

Potential missing files from this Pull request

files commonly committed with a subset of this pr, but not committed this time. (click to collapse)
FilePercentilerate
jest.browser.config.js50.00 %1 out of 2 times
tsconfig.json50.00 %1 out of 2 times
types/util/proto.d.ts50.00 %1 out of 2 times
types/index.d.ts50.00 %1 out of 2 times
README.md50.00 %1 out of 2 times
types/wallet/index.d.ts50.00 %1 out of 2 times
types/util/keyStore.d.ts50.00 %1 out of 2 times
CHANGELOG.md50.00 %1 out of 2 times
test/unit/util/bloom.test.js50.00 %1 out of 2 times
test/unit/chain/index.test.js50.00 %1 out of 2 times

Committed file ranks

  • 80.97%[src/util/utils.js]
  • 68.02%[src/util/bloom.js]
  • 0.00%[src/index.js]
  • 95.95%[dist/aelf.umd.js.map]
  • 99.60%[package.json]
  • 97.57%[dist/aelf.cjs.js]
  • 99.19%[yarn.lock]
  • 97.57%[dist/aelf.umd.js]
  • 95.95%[dist/aelf.cjs.js.map]
  • djouf007 commented 1 year ago

    Same problem, could you help how to fixe that in angular 16 pls, do i need to go to the file js-sha256? is it possible to explain how we do it

    thanks