Ruff: Add and fix TRY300 (#11643)

Co-authored-by: Cody Maffucci <46459665+Maffooch@users.noreply.github.com>

Author: kiblik

dojo/api_v2/views.py

@@ -667,12 +667,12 @@ def update_jira_epic(self, request, pk=None):
                     {"info": "Jira Epic create query sent"},
                     status=status.HTTP_200_OK,
                 )
-            return response
         except ValidationError:
             return Response(
                 {"error": "Bad Request!"},
                 status=status.HTTP_400_BAD_REQUEST,
             )
+        return response
 
 
 # @extend_schema_view(**schema_with_prefetch())

dojo/authorization/roles_permissions.py

@@ -12,9 +12,9 @@ class Roles(IntEnum):
     def has_value(cls, value):
         try:
             Roles(value)
-            return True
         except ValueError:
             return False
+        return True
 
 
 def django_enum(cls):
@@ -129,9 +129,9 @@ class Permissions(IntEnum):
     def has_value(cls, value):
         try:
             Permissions(value)
-            return True
         except ValueError:
             return False
+        return True
 
     @classmethod
     def get_engagement_permissions(cls):

dojo/context_processors.py

@@ -1,3 +1,5 @@
+import contextlib
+
 # import the settings file
 from django.conf import settings
 
@@ -47,12 +49,10 @@ def bind_alert_count(request):
 def bind_announcement(request):
     from dojo.models import UserAnnouncement
 
-    try:
+    with contextlib.suppress(Exception):  # TODO: this should be replaced with more meaningful exception
         if request.user.is_authenticated:
             user_announcement = UserAnnouncement.objects.select_related(
                 "announcement",
             ).get(user=request.user)
             return {"announcement": user_announcement.announcement}
-        return {}
-    except Exception:
-        return {}
+    return {}

dojo/jira_link/helper.py

@@ -406,7 +406,6 @@ def get_jira_connection_raw(jira_server, jira_username, jira_password):
 
         logger.debug("logged in to JIRA %s successfully", jira_server)
 
-        return jira
     except JIRAError as e:
         logger.exception("logged in to JIRA %s unsuccessful", jira_server)
 
@@ -428,6 +427,8 @@ def get_jira_connection_raw(jira_server, jira_username, jira_password):
         add_error_message_to_response("Unable to authenticate to JIRA. Please check the URL, username, password, captcha challenge, Network connection. Details in alert on top right. " + str(error_message))
         raise
 
+    return jira
+
 
 # Gets a connection to a Jira server based on the finding
 def get_jira_connection(obj):
@@ -1185,10 +1186,10 @@ def is_jira_project_valid(jira_project):
     try:
         jira = get_jira_connection(jira_project)
         get_issuetype_fields(jira, jira_project.project_key, jira_project.jira_instance.default_issue_type)
-        return True
     except JIRAError:
         logger.debug("invalid JIRA Project Config, can't retrieve metadata for '%s'", jira_project)
         return False
+    return True
 
 
 def jira_attachment(finding, jira, issue, file, jira_filename=None):
@@ -1209,11 +1210,11 @@ def jira_attachment(finding, jira, issue, file, jira_filename=None):
                 # read and upload a file
                 with open(file, "rb") as f:
                     jira.add_attachment(issue=issue, attachment=f)
-            return True
         except JIRAError as e:
             logger.exception("Unable to add attachment")
             log_jira_alert("Attachment: " + e.text, finding)
             return False
+        return True
     return None
 
 
@@ -1263,11 +1264,11 @@ def close_epic(eng, push_to_jira, **kwargs):
                 if r.status_code != 204:
                     logger.warning(f"JIRA close epic failed with error: {r.text}")
                     return False
-                return True
             except JIRAError as e:
                 logger.exception("Jira Engagement/Epic Close Error")
                 log_jira_generic_alert("Jira Engagement/Epic Close Error", str(e))
                 return False
+            return True
         return None
     add_error_message_to_response("Push to JIRA for Epic skipped because enable_engagement_epic_mapping is not checked for this engagement")
     return False
@@ -1304,15 +1305,15 @@ def update_epic(engagement, **kwargs):
             if (epic_priority := kwargs.get("epic_priority")) is not None:
                 jira_issue_update_kwargs["priority"] = {"name": epic_priority}
             issue.update(**jira_issue_update_kwargs)
-            return True
         except JIRAError as e:
             logger.exception("Jira Engagement/Epic Update Error")
             log_jira_generic_alert("Jira Engagement/Epic Update Error", str(e))
             return False
-    else:
-        add_error_message_to_response("Push to JIRA for Epic skipped because enable_engagement_epic_mapping is not checked for this engagement")
 
-        return False
+        return True
+
+    add_error_message_to_response("Push to JIRA for Epic skipped because enable_engagement_epic_mapping is not checked for this engagement")
+    return False
 
 
 @dojo_model_to_id
@@ -1359,7 +1360,6 @@ def add_epic(engagement, **kwargs):
                 engagement=engagement,
                 jira_project=jira_project)
             j_issue.save()
-            return True
         except JIRAError as e:
             # should we try to parse the errors as JIRA is very strange in how it responds.
             # for example a non existent project_key leads to "project key is required" which sounds like something is missing
@@ -1376,9 +1376,11 @@ def add_epic(engagement, **kwargs):
             log_jira_generic_alert("Jira Engagement/Epic Creation Error",
                                    message + error)
             return False
-    else:
-        add_error_message_to_response("Push to JIRA for Epic skipped because enable_engagement_epic_mapping is not checked for this engagement")
-        return False
+
+        return True
+
+    add_error_message_to_response("Push to JIRA for Epic skipped because enable_engagement_epic_mapping is not checked for this engagement")
+    return False
 
 
 def jira_get_issue(jira_project, issue_key):
@@ -1415,10 +1417,10 @@ def add_comment(obj, note, *, force_push=False, **kwargs):
                 jira.add_comment(
                     j_issue.jira_id,
                     f"({note.author.get_full_name() or note.author.username}): {note.entry}")
-                return True
             except JIRAError as e:
                 log_jira_generic_alert("Jira Add Comment Error", str(e))
                 return False
+            return True
         return None
     return None
 
@@ -1437,10 +1439,10 @@ def add_simple_jira_comment(jira_instance, jira_issue, comment):
         jira.add_comment(
             jira_issue.jira_id, comment,
         )
-        return True
     except Exception as e:
         log_jira_generic_alert("Jira Add Comment Error", str(e))
         return False
+    return True
 
 
 def jira_already_linked(finding, jira_issue_key, jira_id) -> Finding | None:

dojo/models.py

@@ -3085,9 +3085,9 @@ def has_github_issue(self):
         try:
             # Attempt to access the github issue if it exists. If not, an exception will be caught
             _ = self.github_issue
-            return True
         except GITHUB_Issue.DoesNotExist:
             return False
+        return True
 
     def github_conf(self):
         try:

dojo/templatetags/get_banner.py

@@ -23,6 +23,6 @@ def get_banner_conf(attribute):
                     attributes=allowed_attributes,
                     css_sanitizer=CSSSanitizer(allowed_css_properties=["color", "font-weight"])))
             return value
-        return False
     except Exception:
         return False
+    return False

dojo/tools/anchore_enterprise/parser.py

@@ -114,11 +114,11 @@ def policy_name(policies, policy_id):
 def extract_vulnerability_id(trigger_id):
     try:
         vulnerability_id, _ = trigger_id.split("+", 2)
-        if vulnerability_id.startswith("CVE"):
-            return vulnerability_id
-        return None
     except ValueError:
         return None
+    if vulnerability_id.startswith("CVE"):
+        return vulnerability_id
+    return None
 
 
 def search_filepath(text):

dojo/tools/anchorectl_policies/parser.py

@@ -116,11 +116,11 @@ def policy_name(policies, policy_id):
 def extract_vulnerability_id(trigger_id):
     try:
         vulnerability_id, _ = trigger_id.split("+", 2)
-        if vulnerability_id.startswith("CVE"):
-            return vulnerability_id
-        return None
     except ValueError:
         return None
+    if vulnerability_id.startswith("CVE"):
+        return vulnerability_id
+    return None
 
 
 def search_filepath(text):

dojo/tools/api_sonarqube/importer.py

@@ -323,8 +323,6 @@ def import_hotspots(self, test):
                 )
                 items.append(find)
 
-            return items
-
         except Exception as e:
             logger.exception("SonarQube API import issue")
             create_notification(
@@ -336,6 +334,8 @@ def import_hotspots(self, test):
                 obj=test.engagement.product,
             )
 
+        return items
+
     @staticmethod
     def clean_rule_description_html(raw_html):
         search = re.search(

dojo/tools/kubescape/parser.py

@@ -42,10 +42,11 @@ def parse_resource_id(self, resource_id):
             parts = resource_id.split("/")
             resource_type = parts[-2]
             resource_name = parts[-1]
-            return resource_type, resource_name
         except IndexError:
             return None, None
 
+        return resource_type, resource_name
+
     def get_findings(self, filename, test):
         findings = []
         try: