LNP-BP / LNPBPs

LNP/BP standards for bitcoin layer 2 & 3 protocols
https://standards.lnp-bp.org
203 stars 40 forks source link

RGB20: reduce asset name length limit #74

Open zoedberg opened 3 years ago

zoedberg commented 3 years ago

Currently there's an enforcement at schema level of 256 chars on the asset name.

This limit may be too high, so let's consider lowering it a bit.

fedsten commented 3 years ago

Which potential issues do you see by keeping the limit at the current level?

zoedberg commented 3 years ago

Which potential issues do you see by keeping the limit at the current level?

I think that usually asset names will be short (e.g. ticker: "USDT" -> name: "USD Tether").

IMO allowing 256 chars can't cause dangerous issues, but from an interface perspective, every developer in every software that wants to show that info, will have to consider that potentially someone created an asset with a very long name. That's not a funny thing to deal with, especially if most of asset names are short and just one isn't.

So I think we need a reason to keep it so long, not the opposite.

My suggestion is to use 32 chars, it should be enough.

fedsten commented 3 years ago

Fair point, I agree with the proposal of reducing the chars limit to 32

St333p commented 3 years ago

I agree, a 32 chars limit helps to clarify the intended purpose of the "name" field.

weedcoder commented 3 years ago

in liquid, there is no asset name. if you want to have a name, you have to register against blockstream registry and have some cryptographic proofs in a https served .well-know files. you can assign asset name manually in the client configuration file, not really practical. here is the documentation : https://docs.blockstream.com/liquid/developer-guide/proof-of-issuance.html

nicbus commented 3 years ago

the answer might be 42 :smirk:

dr-orlovsky commented 3 years ago

I think we need 37

image