Invalid Felt Regex validation

[arkoc]

Steps to reproduce

1. Go to: https://dwiekawki.github.io/beerus-wasm-demo/
2. Try to execute this:

{
  "execute": {
    "calldata": [],
    "contract_address": "0x49d36570d4e46f48e99674bd3fcc84644ddd6b96f7c741b1562b82f9e004dc7",
    "entry_point_selector": "0x01557182e4359a1f0c6301278e8f5b35a776ab58d39892581e357578fb287836"
  }
}

This is calling total_supply() function.

Response:

{
  "error": "failed to parse request: Error('Felt value does not match regex: 0x01557182e4359a1f0c6301278e8f5b35a776ab58d39892581e357578fb287836', line: 1, column: 194)"
}

Also a side note, when any of functions are returning null (0x) it is not parsed correctly as well.

[dwiekawki]

Hi @arkoc , thank you for describing the issue.

The regex is actually from the official RPC spec, so it can't be invalid by definition. Leading zeros are explicitly forbidden: https://github.com/starkware-libs/starknet-specs/blob/76bdde23c7dae370a3340e40f7ca2ef2520e75b9/api/starknet_api_openrpc.json#L1338

The problem is that the regex is not respected, but this can be easily mitigated on the Beerus side quite easily.

I will post updates here in the issue comments, so don't worry this should be a quick one.

[arkoc]

Thanks for clarification @dwiekawki. We assumed that the issue was the regex, because when we removed it, everything worked.

[dwiekawki]

Fixed version is deployed to https://dwiekawki.github.io/beerus-wasm-demo/.
Release 0.7.1 job is already running.
Enjoy!