refactor: using the hathor prefix on verifyMessage

HathorNetwork / hathor-wallet-service-old

MIT License

4 stars 4 forks source link

refactor: using the hathor prefix on verifyMessage #426

Closed andreabadesso closed 1 year ago

andreabadesso commented 1 year ago

Acceptance Criteria

After https://github.com/HathorNetwork/hathor-wallet-lib/pull/546 is merged, we need to expect the new message prefix on the wallet-service to verify signatures

Security Checklist

[X] Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

codecov[bot] commented 1 year ago

Codecov Report

Merging #426 (b2b2c80) into master (a3a71a4) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #426   +/-   ##
=======================================
  Coverage   95.60%   95.60%           
=======================================
  Files          40       40           
  Lines        2458     2458           
  Branches      278      278           
=======================================
  Hits         2350     2350           
  Misses        108      108

Impacted Files	Coverage Δ
src/utils.ts	`94.17% <100.00%> (ø)`