search

hansmach1ne / LFImap

Local File Inclusion discovery and exploitation tool
Apache License 2.0
197 stars 29 forks source link

'black' linter #54

Closed nrathaus closed 1 month ago

nrathaus commented 1 month ago

The code currently is written with somewhat weird tabbing/ missing of string concating, ' instead of ", may I recommend using black to just look more readable?

Here is a PR that will show you the difference (I just ran 'black' on the root folder) https://github.com/hansmach1ne/LFImap/pull/53

nrathaus commented 1 month ago

Another thing worth working on is f-string from this:

print(
                    "\n"
                    + colors.lightblue("[i]")
                    + " Parsing URL ["
                    + str(iteration + 1)
                    + "/"
                    + str(len(config.parsedUrls))
                    + "]: '"
                    + url
                    + "'"
                )

To: print(f"\n{colors.lightblue('[i]')} Parsing URL [{str(iteration + 1)}/{str(len(config.parsedUrls))}]: '{url}'")

hansmach1ne commented 1 month ago

Code is formatted with 'black' tool: https://github.com/hansmach1ne/LFImap/commit/98d3d974f062a11a715f7c7556ed84533e6600a7

Thanks for the issue.