Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Implement recommendations from 5.0 audit Phase 1B #4502

Merged
merged 26 commits into from
Aug 4, 2023

Conversation

ernestognw
Copy link
Member

@ernestognw ernestognw commented Aug 2, 2023

Full audit report will be out later.

Tracking

Fixes LIB-985
Fixes LIB-987
Fixes LIB-988
Fixes LIB-989
Fixes LIB-990
Fixes LIB-994
Fixes LIB-995
Fixes LIB-1007
Fixes LIB-1006
Fixes LIB-1005
Fixes LIB-1004
Fixes LIB-1001
Fixes LIB-1003
Fixes LIB-1010
Fixes LIB-998
Fixes LIB-1000

@changeset-bot
Copy link

changeset-bot bot commented Aug 2, 2023

🦋 Changeset detected

Latest commit: 8cb5200

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
openzeppelin-solidity Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ernestognw ernestognw force-pushed the audit-1b branch 4 times, most recently from 6dd0a34 to 0460f5a Compare August 2, 2023 06:47
@ernestognw ernestognw requested a review from Amxx August 2, 2023 06:47
@ernestognw
Copy link
Member Author

Force pushes are to make it easier to navigate each fix commit.

.changeset/fifty-owls-retire.md Outdated Show resolved Hide resolved
contracts/metatx/ERC2771Context.sol Outdated Show resolved Hide resolved
contracts/metatx/ERC2771Context.sol Outdated Show resolved Hide resolved
contracts/metatx/ERC2771Forwarder.sol Outdated Show resolved Hide resolved
contracts/metatx/ERC2771Forwarder.sol Show resolved Hide resolved
contracts/proxy/utils/UUPSUpgradeable.sol Outdated Show resolved Hide resolved
contracts/proxy/utils/UUPSUpgradeable.sol Outdated Show resolved Hide resolved
contracts/utils/Address.sol Outdated Show resolved Hide resolved
test/utils/Address.test.js Outdated Show resolved Hide resolved
contracts/finance/VestingWallet.sol Show resolved Hide resolved
This was referenced Sep 10, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants