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

bug: transaction_index serialized to Null #459

Closed
ftupas opened this issue Aug 18, 2023 · 0 comments · Fixed by #466
Closed

bug: transaction_index serialized to Null #459

ftupas opened this issue Aug 18, 2023 · 0 comments · Fixed by #466
Assignees
Labels
bug Inconsistencies or issues which will cause a problem for users or implementors.

Comments

@ftupas
Copy link
Contributor

ftupas commented Aug 18, 2023

Bug Report

Kakarot RPC version: 9828d4a

Current behavior:

As describe in this issue paradigmxyz/reth#4260, transaction_index gets serialized to Null so when we run forge scripts, we get an error. The plan is to make it into u64 in this pr so we will have to set it to 0 for now and write a todo to have an accurate value for it.

Expected behavior:

Should serialize/deserialize properly and forge script runs successfully.

Steps to reproduce:

  1. make katana-rpc-up
  2. forge script scripts/PlainOpcodes.s.sol --rpc-url http://127.0.0.1:3030 --broadcast

Related code:

Failure on receiving a receipt for 0x03ce61d5642cbb216ce62dbabdc5ad84c290760dbbc138cd9683e97c8e452949:
Deserialization Error: invalid type: null, expected a (both 0x-prefixed or not) hex string or byte array containing between (0; 8] bytes at line 1 column 111. Response: 
{
  "transactionHash": "0x03ce61d5642cbb216ce62dbabdc5ad84c290760dbbc138cd9683e97c8e452949",
  "transactionIndex": null,
  "blockHash": "0x02c9e63a66f5b91714f9b132df3cce6da9866e5c1d6ee54598c555a202b2101b",
  "blockNumber": "0xe",
  "from": "0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266",
  "to": null,
  "cumulativeGasUsed": "0xf4240",
  "gasUsed": "0x7a120",
  "contractAddress": "0x5fbdb2315678afecb367f032d93f642f64180aa3",
  "logs": [],
  "logsBloom": "0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000",
  "status": "0x1",
  "effectiveGasPrice": "0xf4240",
  "type": "0x0"
}

Other information:

@ftupas ftupas added the bug Inconsistencies or issues which will cause a problem for users or implementors. label Aug 18, 2023
@ftupas ftupas changed the title bug: bug: transaction_index serialized to Null Aug 18, 2023
@ftupas ftupas self-assigned this Aug 21, 2023
@ftupas ftupas mentioned this issue Aug 21, 2023
11 tasks
@bajpai244 bajpai244 moved this from 🆕 Backlog to 👀 In review in Kakarot's Project Management Aug 22, 2023
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Kakarot's Project Management Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Inconsistencies or issues which will cause a problem for users or implementors.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant