matro7sh / Yelaa

Obtain a clean-cut architecture at the launch of a mission and run some tests ( subdomain search, fuzzing, make screenshots on a given perimeter )
25 stars 4 forks source link

[BUG] - Clean files into .yelaa directory #59

Closed jenaye closed 2 years ago

jenaye commented 2 years ago

currently when running an A-scan, in a and then a B scan, it takes the content of the A scan, and so does A+B, it would have to empty the osint files etc

djnnvx commented 2 years ago

If we change this variable into something that accepts a parameter (IP or URL), we can have a directory named .yelaa/scan-a and another one named .yelaa/scan-b with minimal change.

I am thinking of an optional parameter to YelaaPath (which should be turned into a function). If no argument is specified, we just return .yelaa, otherwise, we append / and the parameter specified.

Maybe it should also create the directory here if it does not exist, or it could be done in a separate function.

darkweak commented 2 years ago

The contents must be under the target URL (AFAIK sqlmap does that).
IMHO introduce a configurable .yelaa path would be good only if we let them to chose the whole path instead of a relative path to the .yelaa directory @bogdzn.

djnnvx commented 2 years ago

That's a good point, but then if we let them chose a specific path (I'm assuming this config is optionnal), what should be the default behaviour ? @darkweak

darkweak commented 2 years ago

We have to write data under {yelaa configured path}/{target url} (e.g. /home/user/dummy/my-scan/domain.com).
By default it should write in the .yelaa/domain.com.
With that there won't be the A+B scan issue unless the user tries to rescan the same domain.
I think about a dryrun option to remove the existing directory data if it already exists.

djnnvx commented 2 years ago

Ok, so to sum it up:

Is it correct ? I already got started on it, I don't want to miss anything

darkweak commented 2 years ago

I agree 👍.

djnnvx commented 2 years ago

Fixed in #60 ! :)~