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

feature discussion - sessions #21

Open
audibleblink opened this issue Jul 13, 2018 · 1 comment
Open

feature discussion - sessions #21

audibleblink opened this issue Jul 13, 2018 · 1 comment

Comments

@audibleblink
Copy link
Contributor

Hey @ustayready , I've got a GitHub module that I want to submit a PR for but I got to thinking...

Since GitHub users are more technically savvy than the average gmail user, I chose to not downgrade GitHub logins to SMS. This means that, in a best-case scenario, I've got 30 second to steal an entered 2FA token.

My GitHub modules instead logs in with the provided creds/OTP and stores the all 'Set-Cookie' values from a successful authentication. From there, timing is less of an issue. I can pop the session cookies into my browser some hours later and still get access to the target's github account.

My point: What do you think about baking this functionality into credsniper core as opposed to at the module level? Maybe default behaviour or by adding a --sessions option? I'm happy to do it, just asking if it's the sort of direction you'd be OK with taking for CredSniper

@ustayready
Copy link
Owner

This is a great idea. I was hoping to streamline the Gmail module to do something similar but I ran into some inconsistencies with the way Google handled authentication. I didn't have much time then but would love to revisit it now.

So, if I understand you right what you mean is if the --sessions flag is provided that CredSniper knows to auth and capture the session tokens? If so, I suppose that could just be handled in it's current state by using the creds and persisting the session information. The only addition would be to include the flag so people can turn it off if they want (or on depending on what the default is) and capturing the session information in the same way we captured creds/2fa.

I like this idea.

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

No branches or pull requests

2 participants