maurosoria / dirsearch

Web path scanner
11.76k stars 2.29k forks source link

Not validating installed dependencies though installed #1135

Closed hsagnik closed 2 weeks ago

hsagnik commented 2 years ago

What is the current behavior?

Suddenly, dirsearch automatically installs the dependencies from requirements.txt via pip.

What is the expected behavior?

Though the previous https://github.com/maurosoria/dirsearch/pull/1063 fix is good but not enough because it still asks to install the dependencies even after installing once again. The tool is not validating the installed dependencies however it should.

In the end the user should not be asked once the dependencies are installed.

Checker:

maurosoria commented 2 years ago

I've experienced the same issue. @shelld3v any clue?

maurosoria commented 2 years ago

BTW, @hsagnik I recommend you use the docker build instead of installing it manually. At least for the moment, until Python 3.9 implementation gets mainstream in linux distros.

shelld3v commented 2 years ago

@hsagnik Sorry I'm trying to understand your statement here. So the problem here is that you had installed the dependencies but dirsearch reported it as missing, right? I might need some info about your system and environment if you don't mind

hsagnik commented 2 years ago

Absolutely @shelld3v moreover it is asking again and again for the same even the dependencies are installed already. Sure, I am happy to help you further.

shelld3v commented 2 years ago

Thank you for your information @hsagnik. I'm thinking about 2 solutions: improving the dependencies check or allowing users to run dirsearch with missing dependencies (save the option in the future). I'm approaching the first one, so I will need your system and environment information, later if we fail, I will have to do the second option

hsagnik commented 2 years ago

Here is the information on the systems I tested:

The installed version of python:

maurosoria commented 6 months ago

Was this issue resolved?

hsagnik commented 3 months ago

Yes @maurosoria the issue is resolved. Thanks to @shelld3v for helping :)