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

[Keeper] JSON - Gas Consumption #1860

Open
gfanton opened this issue Mar 29, 2024 · 2 comments
Open

[Keeper] JSON - Gas Consumption #1860

gfanton opened this issue Mar 29, 2024 · 2 comments
Assignees
@gfanton
Labels
security Security-sensitive issue
Milestone
⏭️Next after mainnet

Comments

@gfanton
Copy link
Member

gfanton commented Mar 29, 2024

To prevent abuse, JSON marshaling and unmarshaling should include gas consumption.
As mentioned here: #1776 (comment), a custom JSON marshaller/unmarshaller implementation is needed to address this.
@gfanton gfanton added the security Security-sensitive issue label Mar 29, 2024
@gfanton gfanton self-assigned this Mar 29, 2024
@gfanton gfanton mentioned this issue Mar 29, 2024
Open
7 tasks
@linear linear bot removed the security Security-sensitive issue label Sep 11, 2024
@Kouteki Kouteki added this to the 🚀 Mainnet launch milestone Oct 16, 2024
@Kouteki Kouteki added security Security-sensitive issue and removed 🌟 must have 🌟 labels Oct 16, 2024
@moul moul closed this as completed Oct 16, 2024
@Kouteki Kouteki reopened this Oct 18, 2024
@Kouteki Kouteki moved this to Rejected in 🍜 Seoul triage Nov 13, 2024
@Kouteki
Copy link
Contributor

Kouteki commented Nov 22, 2024

Related to #2029

@Kouteki
Copy link
Contributor

Kouteki commented Nov 22, 2024 •
edited
Loading

I'm removing this issue out of the mainnet launch scope, in favor of a simpler approach covered in #2949

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gfanton gfanton

Labels
security Security-sensitive issue
Projects
🍜 Seoul triage
Status: Core
🧙‍♂️gno.land core team
Status: Todo
🚀 main.gno.land launch 🚀
Status: Done
Milestone
⏭️Next after mainnet
Development

No branches or pull requests
4 participants
@moul @gfanton @Kouteki @zivkovicmilos