Fix erroneous URL-encoding of the Maven groupId

nscuro · nscuro · commit 0cf231e64bf7 · 2025-01-31T17:44:53.000+01:00
Fixes #4601 Signed-off-by: nscuro <nscuro@protonmail.com>
diff --git a/src/main/java/org/dependencytrack/tasks/repositories/MavenMetaAnalyzer.java b/src/main/java/org/dependencytrack/tasks/repositories/MavenMetaAnalyzer.java
@@ -76,7 +76,7 @@ public RepositoryType supportedRepositoryType() {
     public MetaModel analyze(final Component component) {
         final MetaModel meta = new MetaModel(component);
         if (component.getPurl() != null) {
-            final String mavenGavUrl = urlEncode(component.getPurl().getNamespace().replaceAll("\\.", "/")) + "/" + urlEncode(component.getPurl().getName());
+            final String mavenGavUrl = urlEncode(component.getPurl().getNamespace()).replaceAll("\\.", "/") + "/" + urlEncode(component.getPurl().getName());
             final String url = String.format(baseUrl + REPO_METADATA_URL, mavenGavUrl);
             try (final CloseableHttpResponse response = processHttpRequest(url)) {
                 if (response.getStatusLine().getStatusCode() == HttpStatus.SC_OK) {
diff --git a/src/test/java/org/dependencytrack/tasks/repositories/MavenMetaAnalyzerTest.java b/src/test/java/org/dependencytrack/tasks/repositories/MavenMetaAnalyzerTest.java
@@ -54,4 +54,5 @@ public void testAnalyzerForScalaComponent() throws Exception {
         Assert.assertNotNull(metaModel.getLatestVersion());
         Assert.assertNotNull(metaModel.getPublishedTimestamp());
     }
+
 }

Original file line number	Diff line number	Diff line change
`@@ -54,4 +54,5 @@ public void testAnalyzerForScalaComponent() throws Exception {`
`54`	`54`	`Assert.assertNotNull(metaModel.getLatestVersion());`
`55`	`55`	`Assert.assertNotNull(metaModel.getPublishedTimestamp());`
`56`	`56`	`}`
	`57`	`+`
`57`	`58`	`}`