adaki2004 commented 2 weeks ago

BEGIN_COMMIT_OVERRIDE refactor(protocol): improve contracts based on suggestions from OpenZeppelin chore(protocol): delete ERC20Airdrop2 contract END_COMMIT_OVERRIDE

(A blue checkmark does not mean it is implemented but rather that it is answered in defender -> which in some cases it is a NOT implement / acknowledge, please check defender response !)

[x] N11
[x] N12
[ ] N13 Missing or misleading documentation (Would love to get help from dev. rels if possible on those).
[x] N14
[x] N15
[x] N16
[x] N17
[x] N18

openzeppelin-code[bot] commented 2 weeks ago

refactor(protocol): improve contracts based on suggestions from OpenZeppelin

Generated at commit: ae7a05793922e23dee655631a34a571c32d02357

🚨 Report Summary

Severity Level Results

Contracts Critical
High
Medium
Low
Note
Total 2
2
0
5
40
49

Dependencies Critical
High
Medium
Low
Note
Total 0
0
0
0
0
0

For more details view the full report in OpenZeppelin Code Inspector

	Severity Level	Results
Contracts	Critical High Medium Low Note Total	2 2 0 5 40 49
Dependencies	Critical High Medium Low Note Total	0 0 0 0 0 0

dantaik commented 2 weeks ago

@davidtaikocha I think we only need to upgrade these deployments:

BridgedERC1155
BridgedERC20
ERC1155Vault
ERC20Vault
SgxVerifier

taikoxyz / taiko-mono

refactor(protocol): improve contracts based on suggestions from OpenZeppelin #16949

refactor(protocol): improve contracts based on suggestions from OpenZeppelin

🚨 Report Summary