461 audit fix 9 - Githubissues

getwax / bls-wallet

Core components to use layer 2 smart contract wallets with the BLS signature scheme

MIT License

178 stars 47 forks source link

461 audit fix 9 #579

Closed JohnGuilding closed 1 year ago

JohnGuilding commented 1 year ago

What is this PR doing?

Implements audit fix 9

How can these changes be manually tested?

run contract tests

Does this PR resolve or contribute to any issues?

contributes to #461

Checklist

[x] I have manually tested these changes
[x] Post a link to the PR in the group chat

Guidelines

If your PR is not ready, mark it as a draft
The resolve conversation button is for reviewers, not authors
- (But add a 'done' comment or similar)

JohnGuilding commented 1 year ago

CI seems to be failing due to some typescript issues in the client module. Doesn't seem related to this PR - got the same failures running clients tests in latest from contract-updates. Can look into this if needed but leaving for now.