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

Output from all johnHandlers should go to the console log #43

Open
shinnok opened this issue Jun 26, 2015 · 5 comments
Open

Output from all johnHandlers should go to the console log #43

shinnok opened this issue Jun 26, 2015 · 5 comments
Assignees
Milestone

Comments

@shinnok
Copy link
Member

shinnok commented Jun 26, 2015

Not just the attack one.

@shinnok shinnok added this to the 2.1 milestone Jun 30, 2015
@mathsoft-dev
Copy link
Collaborator

Don't you think it's gonna get confusing if we put attack, John show, password guessing, version checker, show types(hash type checker) in the same textbox? We'll get 2 lines from attack, n Line for version checker, n for thé hash type checker, 2 lines from attack etc... The log might become harder to interpret / understand

@shinnok
Copy link
Member Author

shinnok commented Jul 15, 2015

I know, but I'm not sure how we can make that better. One thing is clear though, we do need output from all of the handlers, for debugging, transparency and during development.

All I can think of right now is to wait for all output to be ready(instead of using readyRead), when the handler has finished, for all except that main cracking one and send in one shot to the Console Log.

@shinnok
Copy link
Member Author

shinnok commented Jul 15, 2015

Maybe make the Console log main view into a QTabWidget with two tabs:

  1. Attack log (output from the johnAttack handler only)
  2. Handlers log (output from all the other handlers, sent to the console log only when handler has finished, thus all output, so that they don't get mangled)

@shinnok shinnok modified the milestones: 2.2, 2.1 Jul 24, 2015
@shinnok shinnok assigned shinnok and unassigned mathsoft-dev Aug 13, 2015
@shinnok
Copy link
Member Author

shinnok commented Aug 13, 2015

I'll take care of this.

@shinnok shinnok modified the milestones: 2.3, 2.2 Aug 20, 2015
@shinnok
Copy link
Member Author

shinnok commented Aug 20, 2015

Moved to 2.3 milestone since this is not high priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants