DefinetlyNotAI/Pwn-Bot

View on GitHub

Showing 10 of 10 total issues

Function crack has a Cognitive Complexity of 68 (exceeds 5 allowed). Consider refactoring.
Open

                                    def crack(filename):
                                        """
                                        Attempts to crack a pcap file using various tools and techniques.

                                        Parameters:
Severity: Minor
Found in Discord_BOT/Bot.py - About 1 day to fix

Cognitive Complexity

Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

A method's cognitive complexity is based on a few simple rules:

  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
  • Code is considered more complex for each "break in the linear flow of the code"
  • Code is considered more complex when "flow breaking structures are nested"

Further reading

File Bot.py has 550 lines of code (exceeds 250 allowed). Consider refactoring.
Open

#!/usr/bin/python
# coding:UTF-8

import json
import os
Severity: Major
Found in Discord_BOT/Bot.py - About 1 day to fix

    Function __send_pcap_files_to_discord has a Cognitive Complexity of 19 (exceeds 5 allowed). Consider refactoring.
    Open

        def __send_pcap_files_to_discord(self, webhook_url):
            """
            Sends .pcap files found in the specified paths to a Discord webhook.
    
            Args:
    Severity: Minor
    Found in Pwngotchi_Files/AutoPCAP.py - About 2 hrs to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    File AutoPCAP.py has 265 lines of code (exceeds 250 allowed). Consider refactoring.
    Open

    import requests
    import colorlog
    import json
    import os
    import pwnagotchi
    Severity: Minor
    Found in Pwngotchi_Files/AutoPCAP.py - About 2 hrs to fix

      Function on_epoch has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.
      Open

          def on_epoch(self, agent, epoch, epoch_data):
              """
              A function that handles the handshake process.
              Checks if the filename ends with '.pcap' and if the AutoPcap plugin is running.
              Logs session details, filename, access point, and client station.
      Severity: Minor
      Found in Pwngotchi_Files/AutoPCAP.py - About 1 hr to fix

      Cognitive Complexity

      Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

      A method's cognitive complexity is based on a few simple rules:

      • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
      • Code is considered more complex for each "break in the linear flow of the code"
      • Code is considered more complex when "flow breaking structures are nested"

      Further reading

      Avoid deeply nested control flow statements.
      Open

                                  with open(file_path, "rb") as f:
                                      content = f.read()
                                      # Adjusted to use 'files' parameter directly without specifying 'Content-Type'
                                      response = requests.post(
                                          webhook_url, files={"file": ("file.pcap", content)}
      Severity: Major
      Found in Pwngotchi_Files/AutoPCAP.py - About 45 mins to fix

        Avoid deeply nested control flow statements.
        Open

                                    if platform.system() != "Linux":
                                        colorlog.error(
                                            f"Windows is not supported for cracking. Please use Linux."
                                        )
                                        log.error(
        Severity: Major
        Found in Discord_BOT/Bot.py - About 45 mins to fix

          Consider simplifying this complex logical expression.
          Open

                  if (
                      config is not None
                      and isinstance(config["token"], str)
                      and isinstance(config["channel_id_(for_pcaps)"], int)
                      and isinstance(config["channel_id_(for_logs)"], int)
          Severity: Major
          Found in Discord_BOT/Bot.py - About 40 mins to fix

            Avoid too many return statements within this function.
            Open

                                                        return False
            Severity: Major
            Found in Discord_BOT/Bot.py - About 30 mins to fix

              Avoid too many return statements within this function.
              Open

                                                          return True
              Severity: Major
              Found in Discord_BOT/Bot.py - About 30 mins to fix
                Severity
                Category
                Status
                Source
                Language