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.

Sign up for GitHub

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

Jump to bottom

🔒 Remove the multicall_value_self Function #167

Merged
merged 2 commits into from
Oct 13, 2023
Merged

🔒 Remove the multicall_value_self Function #167

merged 2 commits into from
Oct 13, 2023

Conversation

pcaversaccio
Copy link
Owner

As title. The underlying issue is that the msg.value should not be trusted.

🐶 Cute Animal Picture

image

@pcaversaccio
🔒 Remove the multicall_value_self Function
b4b08be 
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
@pcaversaccio pcaversaccio added the bug 🐛 Something isn't working label Oct 13, 2023
@pcaversaccio pcaversaccio added this to the 0.0.4 milestone Oct 13, 2023
@pcaversaccio pcaversaccio self-assigned this Oct 13, 2023
@pcaversaccio
♻️ Bump Release Number to 0.0.4
fca6dcd 
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
@pcaversaccio pcaversaccio added the dependencies 🔁 Pull requests that update a dependency file label Oct 13, 2023
@pcaversaccio pcaversaccio merged commit 55af6d8 into main Oct 13, 2023
10 checks passed
@pcaversaccio pcaversaccio deleted the fix-multicall branch October 13, 2023 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@pcaversaccio pcaversaccio

Labels
bug 🐛 Something isn't working dependencies 🔁 Pull requests that update a dependency file
Projects
None yet
Milestone
0.0.4
Development

Successfully merging this pull request may close these issues.
1 participant
@pcaversaccio