Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix abi get inputs #1023

Merged
merged 52 commits into from
Dec 15, 2023
Merged

Fix abi get inputs #1023

merged 52 commits into from
Dec 15, 2023

Conversation

ceyonur
Copy link
Collaborator

@ceyonur ceyonur commented Dec 6, 2023

Why this should be merged

Fixes the enforced strict mode in Unpack decoder, which prevents using extra padded bytes in inputs.

More details: https://docs.soliditylang.org/en/latest/abi-spec.html#strict-encoding-mode

How this works

Adds a flag to toggle the strict mode. It is by default enabled, and DUpgrade disables it (strict mode is off in DUpgrade)

How this was tested

Added unit tests

How is this documented

Should be documented with Dupgrade changes

@ceyonur ceyonur self-assigned this Dec 6, 2023
@ceyonur ceyonur requested a review from darioush as a code owner December 6, 2023 23:21
marun
marun previously approved these changes Dec 7, 2023
@ceyonur ceyonur changed the base branch from replace-old-packers to refactor-allowlist December 7, 2023 18:02
@ceyonur ceyonur added this to the v0.5.11 milestone Dec 14, 2023
darioush
darioush previously approved these changes Dec 14, 2023
accounts/abi/abi_extra_test.go Outdated Show resolved Hide resolved
Base automatically changed from refactor-allowlist to master December 15, 2023 21:05
@ceyonur ceyonur dismissed stale reviews from darioush and marun December 15, 2023 21:05

The base branch was changed.

@ceyonur ceyonur enabled auto-merge (squash) December 15, 2023 21:19
@ceyonur ceyonur disabled auto-merge December 15, 2023 21:38
@ceyonur ceyonur enabled auto-merge (squash) December 15, 2023 22:01
@ceyonur ceyonur merged commit fd23949 into master Dec 15, 2023
8 checks passed
@ceyonur ceyonur deleted the fix-abi-get-inputs branch December 15, 2023 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants