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

Don't publish contract data for every transaction #2970

Closed
PhilWindle opened this issue Oct 23, 2023 · 0 comments · Fixed by #4022 or #4037
Closed

Don't publish contract data for every transaction #2970

PhilWindle opened this issue Oct 23, 2023 · 0 comments · Fixed by #4022 or #4037
Assignees
@alexghr

Comments

@PhilWindle
Copy link
Collaborator

PhilWindle commented Oct 23, 2023 •
edited
Loading

It looks like we are publishing 4 bytes of contract data for every transaction, even if the transaction is not a contract deployment.This needs to be fixed.
@PhilWindle PhilWindle added this to A3 Oct 23, 2023
@PhilWindle PhilWindle converted this from a draft issue Oct 23, 2023
@PhilWindle PhilWindle assigned PhilWindle and alexghr and unassigned PhilWindle Jan 3, 2024
@alexghr alexghr moved this from Todo to In Progress in A3 Jan 10, 2024
@alexghr alexghr mentioned this issue Jan 15, 2024
Merged
PhilWindle pushed a commit that referenced this issue Jan 16, 2024
@alexghr
fix: do not publish empty contract data (#4022)
3ce3ef7 
This PR fixes the sequencer publishing contract for every transaction.

Fix #2970
@github-project-automation github-project-automation bot moved this from In Progress to Done in A3 Jan 16, 2024
@AztecBot AztecBot mentioned this issue Jan 16, 2024
Merged
michaelelliot pushed a commit to Swoir/noir_rs that referenced this issue Feb 28, 2024
@alexghr
fix: do not publish empty contract data (AztecProtocol#4022)
c2497e0 
This PR fixes the sequencer publishing contract for every transaction.

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

@alexghr alexghr

Labels
None yet
Projects
A3
Archived in project
Milestone
No milestone
2 participants
@alexghr @PhilWindle