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

adds from Chick3nman's hashcat kernel and Python script #4

Merged
merged 2 commits into from
Jul 14, 2019
Merged

adds from Chick3nman's hashcat kernel and Python script #4

merged 2 commits into from
Jul 14, 2019

Conversation

roycewilliams
Copy link
Contributor

@roycewilliams roycewilliams commented Jul 14, 2019

Note that the whitespace changes in sha256.md are due to converting from DOS to Unix line separators to match the other files. And actually credit to @hops for the kernel work as well. 😄

@spaze spaze merged commit 746aa97 into spaze:master Jul 14, 2019
@spaze
Copy link
Owner

spaze commented Jul 14, 2019

Hey, awesome work, thanks! I took the liberty and modified your pull request. I've converted the newlines in my repo (thanks for spotting those CRLFs) so this PR didn't need to change them.

@roycewilliams
Copy link
Contributor Author

All credit to @Chick3nman and @hops for the work - I just supplied some horsepower. And I totally trust your judgment on modifying PRs as needed. :D

@spaze
Copy link
Owner

spaze commented Jul 14, 2019

FYI, one more change, the backslash is not part of the string, just to escape the backtick (it wasn't a magic hash otherwise):

- CGq'v]\`1
+ CGq'v]`1

@roycewilliams
Copy link
Contributor Author

Yikes - good catch, sorry I missed that!

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.

2 participants