[Meta] ChainId

[sydhds]

Intro

ChainId implementation for Massa

Documentation

ChainId idea comes from Etherum (and was required to separate the network between Ethereum & Ethereum Classic)

Ethereum ChainId

• https://github.com/ethereum/EIPs/blob/master/EIPS/eip-155.md
• List of chainId:
  • https://chainid.network/ - https://github.com/ethereum-lists/chains
• Ethereum SC (Solidity) documentation:
  • https://eips.ethereum.org/EIPS/eip-1344
  • https://docs.soliditylang.org/en/latest/units-and-global-variables.html#block-and-transaction-properties

Example code from Umbrella:

function update(
    bytes32[] calldata _priceKeys,
    PriceData[] calldata _priceDatas,
    Signature[] calldata _signatures
) external {
    if (_priceKeys.length != _priceDatas.length) revert ArraysDataDoNotMatch();

    bytes32 priceDataHash = keccak256(abi.encode(getChainId(), address(this), _priceKeys, _priceDatas));
    verifySignatures(priceDataHash, _signatures);

    ...
}

function getChainId() public view returns (uint256 id) {
    // solhint-disable-next-line no-inline-assembly
    assembly {
        id := chainid()
    }
}

Implementation

• Massa node
  • [massa-proto]
    • massalabs/massa-proto#57
  • [massa-proto-rs]
    • massalabs/massa-proto-rs#32
  • [massa-sc-runtime]:
    • implement new abi:
      • [old abi] chainid() -> u64
      • [grpc] chainid_v1() -> u64
      • Gas calibration
    • massalabs/massa-sc-runtime#320
  • [massa]:
    • Modify SecureShare impl
    • implement abi calls
      • Add a new entry in config.toml
    • Add to API:
      • [jsonrpc] add to get_status() (public api)
      • [grpc] add to https://github.com/massalabs/massa-proto/blob/main/proto/commons/massa/model/v1/node.proto
        • Add to PublicStatus (privé) & NodeStatus (privé)
          • Warning -> Add to the end of the struct to be backward compat
    • labnet deploy
      • https://github.com/massalabs/private-deployment/pull/31
    • #4588
• Massa node test tooling
  • [massa-test-framework]:
    • Modify op serialization code in Python https://github.com/massalabs/massa-test-framework/pull/24
  • [massa-functional-tests]:
    • implement a test showcasing invalid transaction replay https://github.com/massalabs/massa-functional-tests/pull/15
  • [massa-unit-tests-src]:
    • massalabs/massa-unit-tests-src#38
• Massa tooling
  • [massa-as-sdk]:
    • implement abi function
    • update doc
    • massalabs/massa-as-sdk#316
  • [as_abi_proto]:
    • OUTDATED - will not be updated
• Massa projects
  • [massa station & massa wallet]
    • Massa Wallet: massalabs/station-massa-wallet#856
    • Massa Station: massalabs/station#1297
  • [massa-web3]:
    • Secure share impl modification (Typescript)
      • eval: 0.5 days
    • Risk: Ben is in holidays next week (until Thursday)
    • Next version: 2.3.0?
    • massalabs/massa-web3#510
  • [umbrella]:
    • Use chainId abi in SC in update function
    • Update Certik (Damir)
    • eval: 0.5 days
      • https://github.com/massalabs/umbrella/pull/22
  • [massa-explorer]
    • https://github.com/massalabs/massa-explorer => test.massa.net/v1
    • https://github.com/massalabs/massa.net => test.massa.net
  • Other:
    • [bearby]:
      • Contact Discord channel: #bearby
      • Github issue / PR
      • bearby-wallet/bearby-extension#56 (bearby extension)
      • eval: 1.5 days

Remaining questions

• Should we implement a NetworkId as well?
  • https://besu.hyperledger.org/stable/public-networks/concepts/network-and-chain-id

[damip]

• Update doc https://docs.massa.net/docs/learn/operation-format-execution#signature
  • And Update API JSON et GRPC + ABI

[sydhds]

• massa client: check that op can be emitted
• test.massa.net - https://test.massa.net/v1/ -> wallet in js code

[sydhds]

• PR to chain id list site