No patch url for allura ( sourceforge.net , forge-allura.apache.org )

Signed-off-by: ziad hany <ziadhany2016@gmail.com>

Author: ziadhany

src/packageurl/contrib/purl2url.py

@@ -238,7 +238,7 @@ def build_gitlab_repo_url(purl):
     (r"sourceforge\.net", r"forge-allura\.apache\.org"): {
         "repo_url": "https://{namespace}/{name}",
         "commit_url": "https://{namespace}/{name}/ci/{version}",
-        "patch_url": "https://{namespace}/{name}/ci/{version}",  # FIXME
+        "patch_url": None,  # Can't find a direct patch URL
     },
     # gitweb
     (
@@ -307,7 +307,6 @@ def build_generic_commit_url(purl):
             return template_url["commit_url"].format(
                 namespace=namespace, name=name, version=version
             )
-    return
 
 
 @patch_router.route("pkg:generic/.*")
@@ -328,8 +327,9 @@ def build_generic_patch_url(purl):
             if not re.match(pattern, namespace):
                 continue
 
-            return template_url["patch_url"].format(namespace=namespace, name=name, version=version)
-    return
+            patch_template = template_url.get("patch_url")
+            if patch_template:
+                return patch_template.format(namespace=namespace, name=name, version=version)
 
 
 @commit_router.route("pkg:gitlab/.*", "pkg:bitbucket/.*", "pkg:github/.*")

tests/contrib/test_purl2url.py

@@ -186,16 +186,17 @@ def test_purl2url_get_patch_url():
         "pkg:generic/aomedia.googlesource.com/libavifinfo@43716e9c34d3389b4882fbd1a81c04543ed04fe3": "https://aomedia.googlesource.com/libavifinfo/+/43716e9c34d3389b4882fbd1a81c04543ed04fe3^!?format=TEXT",
         "pkg:generic/chromium.googlesource.com/infra/infra@9ad7ef740dc39834a88bf95c69f35f18b8f45543": "https://chromium.googlesource.com/infra/infra/+/9ad7ef740dc39834a88bf95c69f35f18b8f45543^!?format=TEXT",
         "pkg:generic/android.googlesource.com/device/generic/vulkan-cereal@240dedcb0fa917b3d2dcc4a9d4c332697c5e48a0": "https://android.googlesource.com/device/generic/vulkan-cereal/+/240dedcb0fa917b3d2dcc4a9d4c332697c5e48a0^!?format=TEXT",
-        "pkg:generic/sourceforge.net/p/infrarecorder/code@8fab704119ff23691f075f6a281521b6c7d7e55f": "https://sourceforge.net/p/infrarecorder/code/ci/8fab704119ff23691f075f6a281521b6c7d7e55f",
-        "pkg:generic/sourceforge.net/p/expat/code_git@f0bec73b018caa07d3e75ec8dd967f3785d71bde": "https://sourceforge.net/p/expat/code_git/ci/f0bec73b018caa07d3e75ec8dd967f3785d71bde",
-        "pkg:generic/forge-allura.apache.org/p/allura/git@674e070e5ca7db7c75cf61d8efd2a3e3e49bd946": "https://forge-allura.apache.org/p/allura/git/ci/674e070e5ca7db7c75cf61d8efd2a3e3e49bd946",
+        "pkg:generic/sourceforge.net/p/infrarecorder/code@8fab704119ff23691f075f6a281521b6c7d7e55f": None,
+        "pkg:generic/sourceforge.net/p/expat/code_git@f0bec73b018caa07d3e75ec8dd967f3785d71bde": None,
+        "pkg:generic/forge-allura.apache.org/p/allura/git@674e070e5ca7db7c75cf61d8efd2a3e3e49bd946": None,
         "pkg:generic/gcc.gnu.org/git/gcc@82cc94e5fb69d1c45a386f83798251de5bff9339": "https://gcc.gnu.org/git/?p=gcc.git;a=patch;h=82cc94e5fb69d1c45a386f83798251de5bff9339",
-        "pkg:generic/git.postgresql.org/gitweb/p/infrarecorder/hamn@4f4fed18770ff15da3c7ab1e81854b75181ab5d0": "https://git.postgresql.org/gitweb/p/infrarecorder/?p=hamn.git;a=patch;h=4f4fed18770ff15da3c7ab1e81854b75181ab5d0"
+        "pkg:generic/git.postgresql.org/gitweb/p/infrarecorder/hamn@4f4fed18770ff15da3c7ab1e81854b75181ab5d0": "https://git.postgresql.org/gitweb/p/infrarecorder/?p=hamn.git;a=patch;h=4f4fed18770ff15da3c7ab1e81854b75181ab5d0",
     }
 
     for purl, url in purls_url.items():
         assert url == purl2url.get_patch_url(purl)
 
+
 def test_purl2url_get_commit_url():
     purls_url = {
         "pkg:github/aboutcode-org/vulnerablecode@98e516011d6e096e25247b82fc5f196bbeecff10": "https://github.com/aboutcode-org/vulnerablecode/commit/98e516011d6e096e25247b82fc5f196bbeecff10",
@@ -216,7 +217,7 @@ def test_purl2url_get_commit_url():
         "pkg:generic/sourceforge.net/p/expat/code_git@f0bec73b018caa07d3e75ec8dd967f3785d71bde": "https://sourceforge.net/p/expat/code_git/ci/f0bec73b018caa07d3e75ec8dd967f3785d71bde",
         "pkg:generic/forge-allura.apache.org/p/allura/git@674e070e5ca7db7c75cf61d8efd2a3e3e49bd946": "https://forge-allura.apache.org/p/allura/git/ci/674e070e5ca7db7c75cf61d8efd2a3e3e49bd946",
         "pkg:generic/gcc.gnu.org/git/gcc@82cc94e5fb69d1c45a386f83798251de5bff9339": "https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=82cc94e5fb69d1c45a386f83798251de5bff9339",
-        "pkg:generic/git.postgresql.org/gitweb/p/infrarecorder/hamn@4f4fed18770ff15da3c7ab1e81854b75181ab5d0": "https://git.postgresql.org/gitweb/p/infrarecorder/?p=hamn.git;a=commit;h=4f4fed18770ff15da3c7ab1e81854b75181ab5d0"
+        "pkg:generic/git.postgresql.org/gitweb/p/infrarecorder/hamn@4f4fed18770ff15da3c7ab1e81854b75181ab5d0": "https://git.postgresql.org/gitweb/p/infrarecorder/?p=hamn.git;a=commit;h=4f4fed18770ff15da3c7ab1e81854b75181ab5d0",
     }
 
     for purl, url in purls_url.items():