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

investigate rlp and add tests #1615

Closed
ARR552 opened this issue Feb 1, 2023 · 1 comment · Fixed by #1617
Closed

investigate rlp and add tests #1615

ARR552 opened this issue Feb 1, 2023 · 1 comment · Fixed by #1617
Assignees
@ARR552

Comments

@ARR552
Copy link
Contributor

ARR552 commented Feb 1, 2023

Tx: 0xf86c02844190ab00825208940043d60e87c5dd08c86c3123340705a1556c471987470de4df82000080820b3fa03e5cd16f789cc06e40e19243374ae6f7bf6e9023bb8599c9eaa0bf066206bdaca067afe749b374949fdb1cea34c364c3b125acc2a01b5dc9874e4a125a38fb57b1
This tx is failing when we receive the info from executor.
This is returning this error:

2023-01-29T17:38:50Z	[31mERROR[0m	sequencer/batchbuilder.go:80	failed to process txs, err: %!w(*rlp.decodeError=&{input string too short 0x11aa2e0 [.To (types.LegacyTx)]})%!(EXTRA string=

Add tests to make sure is a problem in the executor.
@ARR552 ARR552 self-assigned this Feb 1, 2023
@ARR552 ARR552 modified the milestone: Synchronization testing Feb 1, 2023
@ARR552 ARR552 mentioned this issue Feb 1, 2023
Merged
@kojnmastyr
Copy link 

ERROR	synchronizer/synchronizer.go:591	error storing trustedBatch. BatchNumber: 12313, BlockNumber: 8208840, error: %!w(*status.Error=&{0xc00000e040})%!(EXTRA string=

Also getting this ^ on v0.0.1-RC10

ARR552 added a commit that referenced this issue Feb 8, 2023
@ARR552
Feature/#1615 rlp (#1617)
9d2b312 
* script

* rlp executor encoding test

* linter

* fix public config

* update prover version

* remove TrustedSequencerURI

* fix script log
tclemos pushed a commit that referenced this issue Feb 14, 2023
@ARR552 @tclemos
Feature/#1615 rlp (#1617)
872315f 
* script

* rlp executor encoding test

* linter

* fix public config

* update prover version

* remove TrustedSequencerURI

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

@ARR552 ARR552

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feature/#1615 rlp 0xPolygonHermez/zkevm-node
2 participants
@ARR552 @kojnmastyr