maurosoria / dirsearch

Web path scanner
11.58k stars 2.29k forks source link

Remove Unused Dependencies #1319

Closed gdrosos closed 7 months ago

gdrosos commented 10 months ago

Summary

This pull request proposes the removal of the unused dependencies: urllib3, cryptography, cffi, idna, chardet from the requirements.txt configuration file. It's part of an ongoing research endeavor focusing on the identification and elimination of code bloat within software projects. Your insights on this would be really valuable.

Rationale

The urllib3 package was added in 913bb40b, the idna and chardet were added on bb5651e. Moreover, cryptography was added on 913bb40, while the dependnecy to cffi was included on 4ab3248.

However, upon analysis of the codebase, it was found that none of the aforementioned packages are currently being directly utilized within the project. Removing those unused dependencies will reduce the overall footprint of the application, mitigate potential security risks, and most importantly, simplify the dependency management process.

Changes

Impact

Testing

All the unit tests pass (29/29)

Requirements


maurosoria commented 7 months ago

Thanks for the detailed explanation!