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

Make fingerprints more pronouncable #8204

Merged
merged 1 commit into from Apr 14, 2022
Merged

Conversation

pali6
Copy link
Member

@pali6 pali6 commented Apr 14, 2022

About the PR

Instead of fingerprints being an MD5 hash this PR postprocesses the hash to 5 groups of 6 letters with consonants and vowels alternating. This should make it easier for people to remember and communicate (parts of) fingerprint IDs.

Before:
image

After:
image

Why's this needed?

Making it so fingerprints are easier to remember and communicate should hopefully make the process of doing forensic work more enjoyable and should slightly reduce the need to paste these into notepad or some other such tool. Also people will have fun when it generates funny words.

Changelog

(u)pali
(*)Fingerprint IDs should now be slightly easier to remember

@pali6 pali6 added the C-QoL A quality of life improvement that makes the game easier to play label Apr 14, 2022
@github-actions github-actions bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Apr 14, 2022
@Yellow-Mushroom
Copy link
Contributor

Absolutely love this. Now instead of referring to a serial door hacker as “6xfa1” they can take on the name “Nocepe” to the entirely of sec

@pali6 pali6 merged commit 7e71549 into goonstation:master Apr 14, 2022
github-actions bot pushed a commit that referenced this pull request Apr 14, 2022
@pali6 pali6 deleted the readable-fprints branch April 14, 2022 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-QoL A quality of life improvement that makes the game easier to play size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants