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

Shortened output and function change #16

Merged
merged 3 commits into from Feb 2, 2021

Conversation

digitalsleuth
Copy link
Contributor

Took a look at the issue raised by @delishouscake (#12) and thought I would combine their thoughts into a pull request to make your work a little easier.
This puts every 'Least Likely' on a separate line, vice a long long of hashes, also removed the period at the end of each as a delimiter is no longer necessary at the end of the line.
Additionally, separated the hash identifier and the hash category with a - for easier reading.
Did not change the colors, as I believe that's more within your purview than mine.

Also, renamed the 'greppable' function to 'grepable' for spelling, and corrected all references to it in the other modules/sections of the tool.

Using the example md5 in the help, final output looks like this:
(keep in mind, this is in a docker environment, colors are not indicative of a normal running shell):

nth-snapshot

Hopefully this helps!

@bee-san
Copy link
Member

bee-san commented Feb 2, 2021

Hey!

Thanks so much for this PR! Let me checkout and try it.

The problem with line by line is:
image

The least likely possible appears on the terminal screen, whereas the user has to scroll up to see the most likely.

Whereas currently:
image

(Note: Same terminal size)

I wanted least likely to be a small dump of everything possible, but most likely to contain the actual useful information. If I could, I'd get rid of least likely but you never know -- it could be RAdmin v2.x!

If you input a file, you'll end up with so many lines your terminal may just cut them off 😂

The rest of the PR is 100% fine though, thank you so much for contributing! :D The name changes to HC and JtR will make the big blob of text much smaller 🤍

PS: Nice to see the Docker image is working! It's on my todo list to upload to Dockerhub ✍️

name_that_hash/prettifier.py Outdated Show resolved Hide resolved
name_that_hash/prettifier.py Outdated Show resolved Hide resolved
@bee-san
Copy link
Member

bee-san commented Feb 2, 2021

@all-contributors please add @digitalsleuth for code

@allcontributors
Copy link
Contributor

@bee-san

I've put up a pull request to add @digitalsleuth! 🎉

@bee-san
Copy link
Member

bee-san commented Feb 2, 2021

Thank you <3 <3 <3 :D Are you in our Discord?

@bee-san bee-san merged commit e133ee4 into HashPals:main Feb 2, 2021
@digitalsleuth
Copy link
Contributor Author

You're welcome! I'm not in the Discord yet, but it seems like something I should do :) You should see me there shortly!

@digitalsleuth digitalsleuth deleted the shortened_output branch February 2, 2021 15:39
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.

None yet

2 participants