Skip to content
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

[Discussion] std.ZeroAddress #2316

Open
leohhhn opened this issue Jun 10, 2024 · 7 comments · May be fixed by #2401
Open

[Discussion] std.ZeroAddress #2316

leohhhn opened this issue Jun 10, 2024 · 7 comments · May be fixed by #2401

Comments

@leohhhn
Copy link
Contributor

leohhhn commented Jun 10, 2024

Description

Should we have a ZeroAddress constant in Gno which will represent a universal burn address? This should be an address for which nobody can obtain the private key. I've found it missing multiple times when developing.

For example, in the case of GRC20 and other similar tokens, people usually send the tokens they want to burn to a zero address.

This "burn" functionality can be implemented in two ways as I see:

  • Instead of having to provide a burn address to send tokens to, simply deduct the burn value from the caller's balance - simpler, but the responsibility of creating this kind of function is on the developer of the package/realm
  • Have a universal ZeroAddress constant in Gno, for example in the std package, for which everyone knows the private key is not obtainable.

WDYT?

@leohhhn leohhhn mentioned this issue Jun 10, 2024
7 tasks
@linhpn99
Copy link
Contributor

If we prioritize transaction transparency, approach 2 is likely the better method

@thinhnx-var
Copy link
Contributor

@leohhhn :
In my view, we surely need ZeroAddress. This is the basis of tokens. I was suprised when I saw this issue :)
And I also think this implementation should be inspired by ETH 0x00 address.

@moul
Copy link
Member

moul commented Jun 13, 2024

Yes, the feature seems useful and will likely improve readability.

Just considering whether std is the most appropriate place. Perhaps we should explore ways to limit the size of the std API.

@leohhhn
Copy link
Contributor Author

leohhhn commented Jun 13, 2024

We could have a "consts" stdlib or similar for this if we had more constants. Related.

I think std is the right place right now, and as soon as #1695 is merged, we can work on the split of std and have a separate place for it.

@thinhnx-var @linhpn99 - would you be open to doing a bit of research on the best way to do this? One solution is to simply have a constant that is std.Address(""), but possibly there is a better way where we actually utilize the address derivation system to generate something that would be akin of a zero address. It needs to not have a private key or similar. It should also be harmonious with std.Address.IsValid() in some way. Also, please think about the possible use cases and examples, and provide them along with the PR, so that the case for it is even stronger.

If you can, after doing the research, please post your suggested approach either here on in the PR description.

@thinhnx-var
Copy link
Contributor

@leohhhn
yea I am working on this and will update our approach and implementation over here.

@grepsuzette
Copy link
Contributor

Good idea.

Which is best:

  • ZeroAddress
  • AddressZero

AddressZero is more scary.
There's a movie called Patient Zero.

@thinhnx-var
Copy link
Contributor

@grepsuzette
I am thinking about (maybe) the best approach and you are giving me one more thing to put in mind. Cool :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

5 participants