hashed-io / hashed-substrate

Hashed Network - Substrate Implementation of Parachain and Solochain
https://hashed.network
MIT License
9 stars 4 forks source link

Increase PSBTMaxLen and MaxVaultsPerUser constants #426

Closed tlacloc closed 1 year ago

tlacloc commented 1 year ago

Description

This pull request increases the PSBTMaxLen and MaxVaultsPerUser constants from 2048 to 4096 and from 10 to 100 respectively, in addition to increasing OutputDescriptorMaxLen to 4096 to match the other changes. These changes improve the flexibility and scalability of the system by allowing for larger PSBTs and more vaults per user.

Changes Made

Impact

These changes enable larger PSBTs and more vaults per user, which can improve the overall flexibility and scalability of the system. However, these changes may also have implications for other parts of the codebase that rely on the previous values of these constants. Any code that interacts with PSBTMaxLen, MaxVaultsPerUser, or OutputDescriptorMaxLen should be reviewed to ensure compatibility with the new values.

didiermis commented 1 year ago

LGTM :+1: Updating the upper limit of a boundedvec does not require a storage migration.