Add support for Bazel PackageURL

Signed-off-by: Keshav Priyadarshi <[email protected]>

Author: keshav-space

src/main/java/com/github/packageurl/PackageURL.java

@@ -145,13 +145,7 @@ public PackageURL(final String purl) throws MalformedPackageURLException {
             } else {
                 this.subpath = null;
             }
-            // qualifiers are optional - check for existence
-            final String rawQuery = uri.getRawQuery();
-            if (rawQuery != null && !rawQuery.isEmpty()) {
-                this.qualifiers = parseQualifiers(rawQuery);
-            } else {
-                this.qualifiers = null;
-            }
+
             // this is the rest of the purl that needs to be parsed
             String remainder = uri.getRawPath();
             // trim trailing '/'
@@ -169,6 +163,14 @@ public PackageURL(final String purl) throws MalformedPackageURLException {
             }
             this.type = StringUtil.toLowerCase(validateType(remainder.substring(start, index)));
 
+            // qualifiers are optional - check for existence
+            final String rawQuery = uri.getRawQuery();
+            if (rawQuery != null && !rawQuery.isEmpty()) {
+                this.qualifiers = parseQualifiers(this.type, rawQuery);
+            } else {
+                this.qualifiers = null;
+            }
+
             start = index + 1;
 
             // version is optional - check for existence
@@ -231,7 +233,7 @@ public PackageURL(
             throws MalformedPackageURLException {
         this.type = StringUtil.toLowerCase(validateType(requireNonNull(type, "type")));
         this.namespace = validateNamespace(this.type, namespace);
-        this.qualifiers = parseQualifiers(qualifiers);
+        this.qualifiers = parseQualifiers(this.type, qualifiers);
         this.name = validateName(this.type, requireNonNull(name, "name"), this.qualifiers);
         this.version = validateVersion(this.type, version);
         this.subpath = validateSubpath(subpath);
@@ -456,7 +458,7 @@ private static String validateMlflowName(final String name, final Map<String,Str
         }
     }
 
-    private static @Nullable Map<String, String> validateQualifiers(final @Nullable Map<String, String> values)
+    private static @Nullable Map<String, String> validateQualifiers(final String type, final @Nullable Map<String, String> values)
             throws MalformedPackageURLException {
         if (values == null || values.isEmpty()) {
             return null;
@@ -467,6 +469,22 @@ private static String validateMlflowName(final String name, final Map<String,Str
             validateKey(key);
             validateValue(key, entry.getValue());
         }
+
+        switch (type) {
+            case StandardTypes.BAZEL:
+                String defaultRegistry = "https://bcr.bazel.build";
+                String repoURL = values.get("repository_url");
+                String normalized = repoURL.toLowerCase();
+                if (normalized.endsWith("/")) {
+                    normalized = normalized.substring(0, normalized.length() - 1);
+                }
+
+                if (normalized.equals(defaultRegistry)){
+                    values.remove("repository_url");
+                }
+                break;
+        }
+
         return values;
     }
 
@@ -593,7 +611,7 @@ private static void verifyTypeConstraints(String type, @Nullable String namespac
         }
     }
 
-    private static @Nullable Map<String, String> parseQualifiers(final @Nullable Map<String, String> qualifiers)
+    private static @Nullable Map<String, String> parseQualifiers(final String type, final @Nullable Map<String, String> qualifiers)
             throws MalformedPackageURLException {
         if (qualifiers == null || qualifiers.isEmpty()) {
             return null;
@@ -606,14 +624,14 @@ private static void verifyTypeConstraints(String type, @Nullable String namespac
                             TreeMap::new,
                             (map, value) -> map.put(StringUtil.toLowerCase(value.getKey()), value.getValue()),
                             TreeMap::putAll);
-            return validateQualifiers(results);
+            return validateQualifiers(type, results);
         } catch (ValidationException ex) {
             throw new MalformedPackageURLException(ex.getMessage());
         }
     }
 
     @SuppressWarnings("StringSplitter") // reason: surprising behavior is okay in this case
-    private static @Nullable Map<String, String> parseQualifiers(final String encodedString)
+    private static @Nullable Map<String, String> parseQualifiers(final String type, final String encodedString)
             throws MalformedPackageURLException {
         try {
             final TreeMap<String, String> results = Arrays.stream(encodedString.split("&"))
@@ -631,7 +649,7 @@ private static void verifyTypeConstraints(String type, @Nullable String namespac
                                 }
                             },
                             TreeMap::putAll);
-            return validateQualifiers(results);
+            return validateQualifiers(type, results);
         } catch (ValidationException e) {
             throw new MalformedPackageURLException(e);
         }
@@ -746,6 +764,12 @@ public static final class StandardTypes {
          * @since 2.0.0
          */
         public static final String APK = "apk";
+        /**
+         * Bazel-based packages.
+         *
+         * @since 2.0.0
+         */
+        public static final String BAZEL = "bazel";
         /**
          * Bitbucket-based packages.
          */