Cosmos-sdk: Change blacklist naming

Created on 16 Jun 2020  ·  2Comments  ·  Source: cosmos/cosmos-sdk

Summary

Change blacklist to a neutral term that suggests its function without further reading.

Problem Definition

The function of blacklisted accounts was added as a fix to prevent module supply invariants from breaking during the lifetime of the chain. While the term does have popular usage in the computer science world, it is not neutral ("whitelist/blacklist") and can have negative connotations whether or not it was originally intended to. Furthermore, I find the usage vague in reference to the exact restrictions placed on a module account address.

Proposal

I don't have a replacement term in mind, but it should reference the specific characteristic that module accounts cannot be sent to. Maybe something like DisableRecvList/DenyRecvList or CannotSendToList


For Admin Use

  • [ ] Not duplicate issue
  • [ ] Appropriate labels applied
  • [ ] Appropriate contributors tagged
  • [ ] Contributor assigned/self-assigned
good first issue proposal-accepted

Most helpful comment

All 2 comments

You have my full support in this regard. Should be a trivial thing to tackle 👍

Was this page helpful?
0 / 5 - 0 ratings

Related issues

cwgoes picture cwgoes  ·  3Comments

hendrikhofstadt picture hendrikhofstadt  ·  3Comments

rigelrozanski picture rigelrozanski  ·  3Comments

kevlubkcm picture kevlubkcm  ·  3Comments

mossid picture mossid  ·  3Comments