Change identity raw data values to be generic Get<u32>

[ltfschoen]

Is there an existing issue?

• I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

• This is not a support question.

Motivation

Motivation mentioned here: https://github.com/polkadot-fellows/RFCs/pull/76/files#diff-26a34bd4a30949abd8e394e45855154a61baf508856915c9ebc105726c28e502R13

Request

Change identity raw data values to be generic Get to allow larger lengths instead of restricted to 32 bytes/chars limit.

Solution

Solution as established here polkadot-fellows/RFCs#76

This Data type is currently re-used across both the new Rococo config and partly in Polkadot.
It should be enough to make that 32 there generic as Get for the identity.
Then in a second change you can increase it for the Polkadot runtime config.
So first a PR to polkadot-sdk to make that value configurable and then we can propose the change in the pallet config.

Are you willing to help with this request?

Yes!

[bkchr]

Please go ahead with a pr @ltfschoen

[ltfschoen]

I can't work on this right at the moment, I'm trying to apply for the PBA "Remote Track" cohort to improve my knowledge

[ggwpez]

Okay thanks for letting us know. Will mark as mentor.

[ltfschoen]

so i've run a local node using the Polkadot SDK and exposed RPC port 9944, then if i use the current Polkadot.js Apps UI that uses Polkadot.js api v10.12.4 and do the following, I recreate the error:

• Go to https://polkadot.js.org/apps/?rpc=ws%3A%2F%2F127.0.0.1%3A9944#/extrinsics
• Choose ALICE > identity > setIdentity(info)
• Click "email: Data" > Choose "Raw"
• Paste value abcdefghijklmnopqrstuvwxyz@me.com
• View error

createType(Call):: Call: failed decoding identity.setIdentity:: Struct: failed on args: {"info":"{\"additional\":\"Vec<(Data,Data)>\",\"display\":\"Data\",\"legal\":\"Data\",\"web\":\"Data\",\"riot\":\"Data\",\"email\":\"Data\",\"pgpFingerprint\":\"Option<[u8;20]>\",\"image\":\"Data\",\"twitter\":\"Data\"}"}:: Struct: failed on info: {"additional":"Vec<(Data,Data)>","display":"Data","legal":"Data","web":"Data","riot":"Data","email":"Data","pgpFingerprint":"Option<[u8;20]>","image":"Data","twitter":"Data"}:: Struct: failed on email: {"_enum":{"None":"Null","Raw":"Bytes","BlakeTwo256":"H256","Sha256":"H256","Keccak256":"H256","ShaThree256":"H256"}}:: Data.Raw values are limited to a maximum length of 32 bytes

So I then tried forking that branch of https://github.com/polkadot-js/api

git clone git@github.com:ltfschoen/api.git
cd api
git remote add upstream https://github.com/polkadot-js/api
git fetch upstream v10.12.4:v10.12.4
git checkout v10.12.4

Searched for the snippet of that error text Call: failed decoding throughout that codebase to find that it is triggered here https://github.com/polkadot-js/api/blob/v10.12.4/packages/types/src/generic/Call.ts#L144

But to check that my Get<u32> related identity changes to the Substrate Node will work in Polkadot.js Apps by modifying that version of the Polkadot.js API, it's necessary to first run my own local version of Poladot.js Apps, and customise the Polkadot.js API dependency that it uses so that it uses the latest metadata from my Substrate Node by following the steps described here https://github.com/polkadot-js/api/blob/v10.12.4/packages/types/src/metadata/README.md, which would be quite time consuming.

So I instead tried using SubXT https://github.com/paritytech/subxt to recreate creating that identity > setIdentity extrinsic, but I encountered errors that I didn't understand how to overcome and added TODOs in the code to highlight what i didn't understand. Here's the PR that shows the changes that I made and the error is shown in the subject of the PR ltfschoen/subxt#1