-
Notifications
You must be signed in to change notification settings - Fork 28
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
The use of metadataRpc and possible reduction of size #111
Comments
@TarikGul we should explore just using the call metadata. We can have a function the pulls out the call metadata and just returns a hex string which can then be passed to the offline device |
@emostov Sounds good. What do you mean by |
Just the calls for each pallet; no errors, events, or storage items. |
With |
@will-yijinin Yup exactly! |
I have problems with big metadataRpc too, hope we can have a solution |
@anhcao142 Hey! Yea currently looking into ways to improve the metadata calls. |
Hi @TarikGul is there any update on this. With the metadata, the file supplied to our offline device exceeds 0.5 MB, which is extremely large. The amount of data we store on the QR code, with respect to the QR codes size generates over 200 QR codes! Would be great if any other solution can be found or suggested in this regard?
|
Yea this has been something on my mind for a bit, but not necessarily a priority (More than happy to make it one for txwrapper-core and get the ball rolling on this for a more optimal solution though). You are absolutely correct most of the data included in the metadata is not absolutely necessary to construct the transaction. More or less its just the implementation that we currently use for whats available. I will put some emphasis on moving forward with this though. My idea was to potential contribute to polkadot-js and add a |
@codemaster101 The following PR above, drastically reduces the size of the metadata. Once it goes in I will give you a ping. |
Thanks a lot @TarikGul |
@codemaster101 PR is in, checkout this example, on how to generate the stripped metadata, and registry. The release should be out sometime today - tomorrow. |
Thanks @TarikGul btw we must use both balances and system as the params for metadata generator right? |
Yea, you can technically get away without using |
Hello there,
Our signing device is isolated and stateless. So currently we pass metadataRpc to it every time we sign a transaction. The metadataRpc is approximately 300k in size and it is inconvenient. Can you point out where in your library you utilize metadataRpc for signing transactions?
We would like to reduce the size of metadataRpc passed into the signing device, possibly eliminate it.
The text was updated successfully, but these errors were encountered: