Consolidate GitHub integration into self-contained dojo/github/ package

Collapse dojo/github.py and dojo/github_issue_link/ into a single
dojo/github/ package matching the canonical dojo/url/ reference layout
described in CLAUDE.md (models, admin, services, ui/forms, ui/views,
ui/urls, templates). Backward-compat re-exports left in dojo/models.py
and dojo/forms.py. No behavior change, no migrations.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: Maffooch

dojo/forms.py

@@ -32,6 +32,14 @@
 from dojo.endpoint.utils import endpoint_filter, endpoint_get_or_create, validate_endpoints_to_add
 from dojo.engagement.queries import get_authorized_engagements
 from dojo.finding.queries import get_authorized_findings
+from dojo.github.ui.forms import (  # noqa: F401 -- backward compat
+    DeleteGITHUBConfForm,
+    ExpressGITHUBForm,
+    GITHUB_IssueForm,
+    GITHUB_Product_Form,
+    GITHUBFindingForm,
+    GITHUBForm,
+)
 from dojo.group.queries import get_authorized_groups, get_group_member_roles
 from dojo.jira import services as jira_services
 from dojo.jira.forms import (  # noqa: F401 backward compat
@@ -79,9 +87,6 @@
     Finding_Group,
     Finding_Template,
     General_Survey,
-    GITHUB_Conf,
-    GITHUB_Issue,
-    GITHUB_PKey,
     Global_Role,
     Note_Type,
     Notes,
@@ -2801,42 +2806,6 @@ class Meta:
         fields = ["id"]
 
 
-class GITHUB_IssueForm(forms.ModelForm):
-
-    class Meta:
-        model = GITHUB_Issue
-        exclude = ["product"]
-
-
-class GITHUBForm(forms.ModelForm):
-    api_key = forms.CharField(widget=forms.PasswordInput, required=True)
-
-    class Meta:
-        model = GITHUB_Conf
-        exclude = ["product"]
-
-
-class DeleteGITHUBConfForm(forms.ModelForm):
-    id = forms.IntegerField(required=True,
-                            widget=forms.widgets.HiddenInput())
-
-    class Meta:
-        model = GITHUB_Conf
-        fields = ["id"]
-
-
-class ExpressGITHUBForm(forms.ModelForm):
-    password = forms.CharField(widget=forms.PasswordInput, required=True)
-    issue_key = forms.CharField(required=True, help_text="A valid issue ID is required to gather the necessary information.")
-
-    class Meta:
-        model = GITHUB_Conf
-        exclude = ["product", "epic_name_id", "open_status_key",
-                    "close_status_key", "info_mapping_severity",
-                    "low_mapping_severity", "medium_mapping_severity",
-                    "high_mapping_severity", "critical_mapping_severity", "finding_text"]
-
-
 class Benchmark_Product_SummaryForm(forms.ModelForm):
 
     class Meta:
@@ -3222,23 +3191,6 @@ class Meta:
         # fields = ['selenium_script']
 
 
-class GITHUB_Product_Form(forms.ModelForm):
-    git_conf = forms.ModelChoiceField(queryset=GITHUB_Conf.objects.all(), label="GITHUB Configuration", required=False)
-
-    class Meta:
-        model = GITHUB_PKey
-        exclude = ["product"]
-
-
-class GITHUBFindingForm(forms.Form):
-    def __init__(self, *args, **kwargs):
-        self.enabled = kwargs.pop("enabled")
-        super().__init__(*args, **kwargs)
-        self.fields["push_to_github"] = forms.BooleanField()
-        self.fields["push_to_github"].required = False
-        self.fields["push_to_github"].help_text = "Checking this will overwrite content of your Github issue, or create one."
-
-    push_to_github = forms.BooleanField(required=False)
 
 
 class LoginBanner(forms.Form):

dojo/github/__init__.py

@@ -0,0 +1 @@
+import dojo.github.admin  # noqa: F401

dojo/github/admin.py

@@ -0,0 +1,15 @@
+from django.contrib import admin
+
+from dojo.github.models import (
+    GITHUB_Clone,
+    GITHUB_Conf,
+    GITHUB_Details_Cache,
+    GITHUB_Issue,
+    GITHUB_PKey,
+)
+
+admin.site.register(GITHUB_Conf)
+admin.site.register(GITHUB_Issue)
+admin.site.register(GITHUB_Clone)
+admin.site.register(GITHUB_Details_Cache)
+admin.site.register(GITHUB_PKey)

dojo/github/models.py

@@ -0,0 +1,45 @@
+from django.db import models
+from django.utils.translation import gettext as _
+
+from dojo.models import Finding, Product
+
+
+class GITHUB_Conf(models.Model):
+    configuration_name = models.CharField(max_length=2000, help_text=_("Enter a name to give to this configuration"), default="")
+    api_key = models.CharField(max_length=2000, help_text=_("Enter your Github API Key"), default="")
+
+    def __str__(self):
+        return self.configuration_name
+
+
+class GITHUB_Issue(models.Model):
+    issue_id = models.CharField(max_length=200)
+    issue_url = models.URLField(max_length=2000, verbose_name=_("GitHub issue URL"))
+    finding = models.OneToOneField(Finding, null=True, blank=True, on_delete=models.CASCADE)
+
+    def __str__(self):
+        return str(self.issue_id) + "| GitHub Issue URL: " + str(self.issue_url)
+
+
+class GITHUB_Clone(models.Model):
+    github_id = models.CharField(max_length=200)
+    github_clone_id = models.CharField(max_length=200)
+
+
+class GITHUB_Details_Cache(models.Model):
+    github_id = models.CharField(max_length=200)
+    github_key = models.CharField(max_length=200)
+    github_status = models.CharField(max_length=200)
+    github_resolution = models.CharField(max_length=200)
+
+
+class GITHUB_PKey(models.Model):
+    product = models.ForeignKey(Product, on_delete=models.CASCADE)
+
+    git_project = models.CharField(max_length=200, blank=True, verbose_name=_("Github project"), help_text=_("Specify your project location. (:user/:repo)"))
+    git_conf = models.ForeignKey(GITHUB_Conf, verbose_name=_("Github Configuration"),
+                                 null=True, blank=True, on_delete=models.CASCADE)
+    git_push_notes = models.BooleanField(default=False, blank=True, help_text=_("Notes added to findings will be automatically added to the corresponding github issue"))
+
+    def __str__(self):
+        return self.product.name + " | " + self.git_project

dojo/github.py dojo/github/services.pydojo/github.py renamed to dojo/github/services.py

@@ -1,19 +1,23 @@
-# python
 import logging
 import sys
 
 from django.template.loader import render_to_string
-
-# External libs
 from github import Auth, Github
 
-# Dojo related imports
-from dojo.models import Engagement, GITHUB_Issue, GITHUB_PKey, Product
+from dojo.github.models import GITHUB_Issue, GITHUB_PKey
+from dojo.models import Engagement, Product
 
-# Create global
 logger = logging.getLogger(__name__)
 
 
+def validate_github_credentials(api_key):
+    """Verify a GitHub API key by fetching the authenticated user. Raises on failure."""
+    g = Github(api_key)
+    user = g.get_user()
+    logger.debug("Using user " + user.login)
+    return user.login
+
+
 def reopen_external_issue_github(find, note, prod, eng):
     # Ensure the system setting for GitHub integration is enabled
     from dojo.utils import get_system_setting  # noqa: PLC0415 circular import

dojo/templates/dojo/delete_github.html …github/templates/dojo/delete_github.htmldojo/templates/dojo/delete_github.html renamed to dojo/github/templates/dojo/delete_github.html dojo/templates/dojo/delete_github.html renamed to dojo/github/templates/dojo/delete_github.html

@@ -0,0 +1,58 @@
+from django import forms
+
+from dojo.github.models import GITHUB_Conf, GITHUB_Issue, GITHUB_PKey
+
+
+class GITHUB_IssueForm(forms.ModelForm):
+
+    class Meta:
+        model = GITHUB_Issue
+        exclude = ["product"]
+
+
+class GITHUBForm(forms.ModelForm):
+    api_key = forms.CharField(widget=forms.PasswordInput, required=True)
+
+    class Meta:
+        model = GITHUB_Conf
+        exclude = ["product"]
+
+
+class DeleteGITHUBConfForm(forms.ModelForm):
+    id = forms.IntegerField(required=True,
+                            widget=forms.widgets.HiddenInput())
+
+    class Meta:
+        model = GITHUB_Conf
+        fields = ["id"]
+
+
+class ExpressGITHUBForm(forms.ModelForm):
+    password = forms.CharField(widget=forms.PasswordInput, required=True)
+    issue_key = forms.CharField(required=True, help_text="A valid issue ID is required to gather the necessary information.")
+
+    class Meta:
+        model = GITHUB_Conf
+        exclude = ["product", "epic_name_id", "open_status_key",
+                    "close_status_key", "info_mapping_severity",
+                    "low_mapping_severity", "medium_mapping_severity",
+                    "high_mapping_severity", "critical_mapping_severity", "finding_text"]
+
+
+class GITHUB_Product_Form(forms.ModelForm):
+    git_conf = forms.ModelChoiceField(queryset=GITHUB_Conf.objects.all(), label="GITHUB Configuration", required=False)
+
+    class Meta:
+        model = GITHUB_PKey
+        exclude = ["product"]
+
+
+class GITHUBFindingForm(forms.Form):
+    def __init__(self, *args, **kwargs):
+        self.enabled = kwargs.pop("enabled")
+        super().__init__(*args, **kwargs)
+        self.fields["push_to_github"] = forms.BooleanField()
+        self.fields["push_to_github"].required = False
+        self.fields["push_to_github"].help_text = "Checking this will overwrite content of your Github issue, or create one."
+
+    push_to_github = forms.BooleanField(required=False)