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

introduce pubkey id-suffixes, toggled via /ids command #179

Merged
merged 12 commits into from
May 5, 2020
Merged

Conversation

cblgh
Copy link
Member

@cblgh cblgh commented Apr 25, 2020

this PR adds support for the /ids command (added in a PR to cabal-client).

when /ids is run, the cli shows nicknames appended with the first 9 characters of their public key, see the image below.

9 characters was chosen as a long enough identifier that brute forcing is discouraged (based on numbers from lachenmayer/vanity-dat), even if it is a bit garish.

the intent is to run with /ids on when you want to uniquely identify someone, e.g. when initiating a new PM or issuing a moderation action such as adding someone as a moderator, or banning them.

image

p.s. the PR also fixes command output not being visible when joined to multiple cabals at the same time (only the first joined cabal would have its input shown)

TODO

  • make an attempt at arbitrary collisions
  • invert bg colour of id segment
  • add a keybinding (alt-i/alt-l?) for toggling /ids on off

@cblgh cblgh changed the title introduce 9 character long ids, toggled via /ids command introduce pubkey id-suffixes, toggled via /ids command May 4, 2020
@cblgh cblgh merged commit 275bf4c into master May 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant