Sourcery refactored main branch by sourcery-ai[bot] · Pull Request #1 · RetroPackets/Fugitive

sourcery-ai · 2023-06-09T22:06:51Z

Branch main refactored by Sourcery.

If you're happy with these changes, merge this Pull Request using the Squash and merge strategy.

See our documentation here.

Run Sourcery locally

Reduce the feedback loop during development by using the Sourcery editor plugin:

Review changes via command line

To manually merge these changes, make sure you're on the main branch, then run:

git fetch origin sourcery/main
git merge --ff-only FETCH_HEAD
git reset HEAD^

Help us improve this pull request!

sourcery-ai

Due to GitHub API limits, only the first 60 comments can be shown.

sourcery-ai · 2023-06-09T22:06:54Z

src/adv-search.py

 def parser_error(self):
    banner()
-    print("Usage: python " + sys.argv[0] + " [Options] use -h for help")
+    print(f"Usage: python {sys.argv[0]} [Options] use -h for help")


Function parser_error refactored with the following changes:

Use f-string instead of string concatenation [×2] (use-fstring-for-concatenation)

sourcery-ai · 2023-06-09T22:06:54Z

src/adv-search.py

    parser.add_argument('-o', '--output', required=False)
-    args = parser.parse_args()
-    return args
+    return parser.parse_args()


Function parse_args refactored with the following changes:

Inline variable that is immediately returned (inline-immediately-returned-variable)

sourcery-ai · 2023-06-09T22:06:54Z

src/adv-search.py

-    if (url == []):
+    if not url:


Function urlExtract refactored with the following changes:

Replaces an empty collection equality with a boolean operation (simplify-empty-collection-comparison)

sourcery-ai · 2023-06-09T22:06:54Z

src/adv-search.py

-    if(args.dorkfile):
+    if args.dorkfile:
        with open(dorks) as f:
            while True:
-                line = f.readline()
-                if not line:
+                if not (line := f.readline()):
                    break
-                else:
-                    driver = helium.start_firefox(headless=False)
-                    go_to('https://www.google.com/search?q=site:doxbin.com | site:instagram.com | site:facebook.com | site:youtube.com | site:zerobin.net | site:pastebin.com | site:skidbin.net | site:hastebin.com | site:twitter.com | site:linkedin.com | site:pinterest.com | site:tumblr.com | site:snapchat.com | site:reddit.com | site:github.com | site:gitlab.com | site:pornhub.com | site:whatsapp.com  \ site:tiktok.com "' + test +'"')
-                    input_(line)
-                    time.sleep(5)
-                    flow()
+                driver = helium.start_firefox(headless=False)
+                go_to('https://www.google.com/search?q=site:doxbin.com | site:instagram.com | site:facebook.com | site:youtube.com | site:zerobin.net | site:pastebin.com | site:skidbin.net | site:hastebin.com | site:twitter.com | site:linkedin.com | site:pinterest.com | site:tumblr.com | site:snapchat.com | site:reddit.com | site:github.com | site:gitlab.com | site:pornhub.com | site:whatsapp.com  \ site:tiktok.com "' + test +'"')
+                input_(line)
+                time.sleep(5)
+                flow()

    else:
        line = args.dork
        driver = helium.start_firefox(headless=False)
        time.sleep(5)
-        go_to('https://www.google.com/search?q="' + test + '"')
+        go_to(f'https://www.google.com/search?q="{test}"')


Function exechaha refactored with the following changes:

Use named expression to simplify assignment and conditional (use-named-expression)

Swap if/else branches [×2] (swap-if-else-branches)

Use f-string instead of string concatenation [×2] (use-fstring-for-concatenation)

Remove unnecessary else after guard condition (remove-unnecessary-else)

sourcery-ai · 2023-06-09T22:06:54Z

src/adv-search.py

-    if(args.output):
-        file1 = open(args.output+'.txt', 'w')
-        for i in clean(final):
-            file1.write(i.strip()+'\n')
-        file1.close()
+    if args.output:
+        with open(f'{args.output}.txt', 'w') as file1:
+            for i in clean(final):
+                file1.write(i.strip()+'\n')
    else:
        for i in clean(final):
-            print(f"{Fore.YELLOW}[{Fore.GREEN}ϟ{Fore.YELLOW}] {Fore.BLUE}" + i)
+            print(f"{Fore.YELLOW}[{Fore.GREEN}ϟ{Fore.YELLOW}] {Fore.BLUE}{i}")


Function main refactored with the following changes:

Use with when opening file to ensure closure (ensure-file-closed)

Use f-string instead of string concatenation [×2] (use-fstring-for-concatenation)

sourcery-ai · 2023-06-09T22:06:56Z

src/geo-recon/modules/commands.py

-      print(Fore.WHITE+'# Commands')
-      print('python geo-recon.py --help or -h                   (Display help)')
-      print('python geo-recon.py 138.121.128.19 --nmap or -n    (Nmap standard use)')
-      print('python geo-recon.py 138.121.128.19                 (Standard use, infos about IP)')
-      print('python geo-recon.py --commands or -c               (Display commands availables )') 
+   print(f'{Fore.WHITE}# Commands')
+   print('python geo-recon.py --help or -h                   (Display help)')
+   print('python geo-recon.py 138.121.128.19 --nmap or -n    (Nmap standard use)')
+   print('python geo-recon.py 138.121.128.19                 (Standard use, infos about IP)')
+   print('python geo-recon.py --commands or -c               (Display commands availables )') 


Function listCommand refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

sourcery-ai · 2023-06-09T22:06:57Z

src/geo-recon/modules/getData.py

-    r = requests.get('http://ip-api.com/json/'+syA1)
+    r = requests.get(f'http://ip-api.com/json/{syA1}')
    response = json.loads(r.text)

    #Iterate the data, filter & print ( data for specific fields
-    #print ( sys.argv[1] ) 
-    print ( Fore.WHITE + "Country: "+ response["country"])
+    #print ( sys.argv[1] )
+    print(f"{Fore.WHITE}Country: " + response["country"])


Function getGeo refactored with the following changes:

Use f-string instead of string concatenation [×2] (use-fstring-for-concatenation)

sourcery-ai · 2023-06-09T22:06:57Z

src/geo-recon/modules/myip.py

    response = json.loads(r.text)
-    ip = (response["ip"])
-    return ip
+    return response["ip"]


Function myIp refactored with the following changes:

Inline variable that is immediately returned (inline-immediately-returned-variable)

sourcery-ai · 2023-06-09T22:06:57Z

src/holehe/build/lib/holehe/modules/cms/gravatar.py

-                'FullName': str(FullName)+" / "+str(data['entry'][0]["profileUrl"]),
+                'FullName': f"{str(FullName)} / "
+                + str(data['entry'][0]["profileUrl"])


Function gravatar refactored with the following changes:

Use f-string instead of string concatenation (use-fstring-for-concatenation)

sourcery-ai · 2023-06-09T22:06:57Z

src/holehe/build/lib/holehe/modules/cms/voxmedia.py

-        if rep["available"]:
-            out.append({"name": name,"domain":domain,"method":method,"frequent_rate_limit":frequent_rate_limit,
-                        "rateLimit": False,
-                        "exists": False,
-                        "emailrecovery": None,
-                        "phoneNumber": None,
-                        "others": None})
-        elif rep["message"]=="You cannot use this email address.":
+        if (
+            rep["available"]
+            or rep["message"] == "You cannot use this email address."
+        ):


Function voxmedia refactored with the following changes:

Merge duplicate blocks in conditional (merge-duplicate-blocks)

Remove redundant conditional (remove-redundant-if)

'Refactored by Sourcery'

b8fa138

sourcery-ai bot requested a review from RetroPackets June 9, 2023 22:06

sourcery-ai bot commented Jun 9, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sourcery refactored main branch#1

Sourcery refactored main branch#1
sourcery-ai[bot] wants to merge 1 commit intomainfrom
sourcery/main

sourcery-ai bot commented Jun 9, 2023

Uh oh!

sourcery-ai bot left a comment

Uh oh!

sourcery-ai bot Jun 9, 2023

Uh oh!

sourcery-ai bot Jun 9, 2023

Uh oh!

sourcery-ai bot Jun 9, 2023

Uh oh!

sourcery-ai bot Jun 9, 2023

Uh oh!

sourcery-ai bot Jun 9, 2023

Uh oh!

sourcery-ai bot Jun 9, 2023

Uh oh!

sourcery-ai bot Jun 9, 2023

Uh oh!

sourcery-ai bot Jun 9, 2023

Uh oh!

sourcery-ai bot Jun 9, 2023

Uh oh!

sourcery-ai bot Jun 9, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

0 participants

Conversation

sourcery-ai bot commented Jun 9, 2023

Uh oh!

sourcery-ai bot left a comment

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jun 9, 2023

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jun 9, 2023

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jun 9, 2023

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jun 9, 2023

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jun 9, 2023

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jun 9, 2023

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jun 9, 2023

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jun 9, 2023

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jun 9, 2023

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jun 9, 2023

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

0 participants