Page MenuHomeSolus

The inclusion of hydra fixes T4828
AbandonedPublic

Authored by Devil505 on Aug 11 2018, 12:38 PM.

Details

Reviewers
DataDrake
Group Reviewers
Triage Team
Maniphest Tasks
T4828: THC Hydra
Test Plan

hydra CLI works
xhydra (GTK+2 app) works
I'm not an expert about pentest so if someone can make some tests...

Diff Detail

Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Devil505 created this revision.Aug 11 2018, 12:38 PM
Devil505 requested review of this revision.Aug 11 2018, 12:38 PM
Devil505 updated this revision to Diff 8819.Aug 11 2018, 4:10 PM

Added new depends to enable some supports

DataDrake requested changes to this revision.Aug 18 2018, 11:31 PM
DataDrake added a subscriber: DataDrake.

Please rename this to thc-hydra to follow upstream.

This revision now requires changes to proceed.Aug 18 2018, 11:31 PM
Devil505 updated this revision to Diff 9021.Aug 19 2018, 5:19 AM

Renaming thc-hydra as requested

DataDrake accepted this revision.Aug 19 2018, 1:43 PM

LGTM. Thanks!

This revision is now accepted and ready to land.Aug 19 2018, 1:43 PM
DataDrake requested changes to this revision.Aug 19 2018, 1:45 PM

Spoke too soon: see inline comments

pspec_x86_64.xml
27

This isn't a valid filepath.

These should likely be in /usr/share/thc-hydra

This revision now requires changes to proceed.Aug 19 2018, 1:45 PM

@Devil505 Any intent on updating this patch?

@Devil505 Any intent on updating this patch?

yes, I found a patch to fix the makefile.
I'll also add a .desktop and png icon for xhydra.

JoshStrobl abandoned this revision.Dec 11 2018, 9:30 PM

Abandoning due to lack of action.