search

AcalaNetwork / chopsticks

Create parallel reality of your Substrate network.
Apache License 2.0
133 stars 80 forks source link

hrmp from unregistered parachains #635

Closed ermalkaleci closed 8 months ago

NachoPal commented 8 months ago

It didn't work:

Getting this error:

[11:45:32.039] TRACE (txpool/46745): build block
    app: "chopsticks"
    params: {
      "transactions": [],
      "upwardMessages": {},
      "downwardMessages": [],
      "horizontalMessages": {
        "2000": [
          {
            "sentAt": 6084721,
            "data": "0x000314000400000204320503d2029649000700e87648171300000204320503d2029649000700e876481700060103008c864782380100ac320103d202964900d43593c715fdd31c61141abd04a99fd6822c8558854ccde39a5684e7a56da27d01a10f140d010000010100d43593c715fdd31c61141abd04a99fd6822c8558854ccde39a5684e7a56da27d"
          }
        ]
      }
    }
file:///Users/nacho/Desktop/PARITY/Repos/chopsticks/packages/core/src/blockchain/inherent/parachain/validation-data.ts:119
        const hrmpMessages = _.mergeWith(extrinsic.horizontalMessages, params.horizontalMessages, (objValue, srcValue)=>objValue.concat(srcValue));
                                                                                                                                 ^

TypeError: Cannot read properties of undefined (reading 'concat')
    at file:///Users/nacho/Desktop/PARITY/Repos/chopsticks/packages/core/src/blockchain/inherent/parachain/validation-data.ts:119:130
    at baseMergeDeep (/Users/nacho/Desktop/PARITY/Repos/chopsticks/node_modules/lodash/lodash.js:3685:11)
    at /Users/nacho/Desktop/PARITY/Repos/chopsticks/node_modules/lodash/lodash.js:3645:11
    at /Users/nacho/Desktop/PARITY/Repos/chopsticks/node_modules/lodash/lodash.js:4967:15
    at baseMerge (/Users/nacho/Desktop/PARITY/Repos/chopsticks/node_modules/lodash/lodash.js:3642:7)
    at /Users/nacho/Desktop/PARITY/Repos/chopsticks/node_modules/lodash/lodash.js:13541:7
    at Function.<anonymous> (/Users/nacho/Desktop/PARITY/Repos/chopsticks/node_modules/lodash/lodash.js:4915:13)
    at apply (/Users/nacho/Desktop/PARITY/Repos/chopsticks/node_modules/lodash/lodash.js:489:27)
    at Function.mergeWith (/Users/nacho/Desktop/PARITY/Repos/chopsticks/node_modules/lodash/lodash.js:6627:16)
    at SetValidationData.createInherents (file:///Users/nacho/Desktop/PARITY/Repos/chopsticks/packages/core/src/blockchain/inherent/parachain/validation-data.ts:119:32)
ermalkaleci commented 8 months ago

@NachoPal I've pushed some changes. Now we inject arbitrary paraId into validation data