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

Codetective - find password hashes and CC numbers #75

Open
wants to merge 24 commits into
base: master
Choose a base branch
from

Conversation

apolkosnik-old
Copy link
Contributor

Service using a tool to determine the crypto/encoding algorithm used according to traces from its representation

@apolkosnik-old
Copy link
Contributor Author

Can I ask for a quick review of this? It provides a really nice way to find hashes, credit card numbers, UIDs, and cookies. Thanks in advance.

@mgoffin
Copy link
Contributor

mgoffin commented Feb 27, 2015

Not sure I'm a fan of requiring a git clone into a directory in a git clone (accidental commits of this if someone pushes a PR with a new feature can happen).

Is there any way to design this such that it doesn't require it to be cloned into crits_services and can exist somewhere else?

@apolkosnik
Copy link
Contributor

copy will suffice
On Feb 27, 2015 11:19 AM, "Mike Goffin" [email protected] wrote:

Not sure I'm a fan of requiring a git clone into a directory in a git
clone (accidental commits of this if someone pushes a PR with a new feature
can happen).

Is there any way to design this such that it doesn't require it to be
cloned into crits_services and can exist somewhere else?


Reply to this email directly or view it on GitHub
#75 (comment).

@apolkosnik-old
Copy link
Contributor Author

Updated the install instructions

@mgoffin
Copy link
Contributor

mgoffin commented Feb 27, 2015

I guess I should be more clear.

There shouldn't be any copying/installing/cloning/etc of any files into what is potentially a repository, such as crits_services.

@apolkosnik-old
Copy link
Contributor Author

Ok, I got a go-ahead from the author, and will add what's needed shortly.

@apolkosnik-old
Copy link
Contributor Author

Ok, how about now?

@mgoffin
Copy link
Contributor

mgoffin commented Feb 27, 2015

I took a look at the Codetective stuff and evidently it's just a standalone script with no way to install it (i was really hoping this could just be done via pip install). I really don't like having code like this the way it is. It means if a new Codetective version comes out you have to release a new service version as opposed to just updating your codetective stuff and it "just working".

And with it being GPL and being housed under an MIT licensed service, I could swear there's some special disclaimer you need to make somewhere about something.

@apolkosnik-old
Copy link
Contributor Author

I was trying to avoid the whole LICENSE ordeal by just allowing the users to git-clone that thing into their service folder... and it would have just worked between the updates. Either way, I can backpedal this whole thing or ask the author to jump in here and ask if he'd be willing to relicense or something.

@apolkosnik-old
Copy link
Contributor Author

Is it any better now? I'm asking the users to grab the GPL part themselves, and it's basically a single script that they can update without updating the service.

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.

3 participants