Fix crash after enabling MANIFEST parsing

DreierF · DreierF · commit 4ee95e95472e · 2022-01-03T19:14:57.000+01:00
diff --git a/plugins/license-gather-plugin/src/main/kotlin/com/github/vlsi/gradle/license/GatherLicenseTask.kt b/plugins/license-gather-plugin/src/main/kotlin/com/github/vlsi/gradle/license/GatherLicenseTask.kt
@@ -500,6 +500,10 @@ open class GatherLicenseTask @Inject constructor(
             )
             JarFile(file).use { jar ->
                 val bundleLicense = jar.manifest.mainAttributes.getValue("Bundle-License")
+                if (bundleLicense.startsWith("http://") || bundleLicense.startsWith("https://")) {
+                    // Ignore URLs here as it will fail during parsing
+                    return
+                }
                 val license = bundleLicense?.substringBefore(";")?.let {
                     licenseExpressionParser.parse(it)
                 }

Original file line number	Diff line number	Diff line change
`@@ -500,6 +500,10 @@ open class GatherLicenseTask @Inject constructor(`
`500`	`500`	`)`
`501`	`501`	`JarFile(file).use { jar ->`
`502`	`502`	`val bundleLicense = jar.manifest.mainAttributes.getValue("Bundle-License")`
	`503`	`+ if (bundleLicense.startsWith("http://") \|\| bundleLicense.startsWith("https://")) {`
	`504`	`+ // Ignore URLs here as it will fail during parsing`
	`505`	`+ return`
	`506`	`+ }`
`503`	`507`	`val license = bundleLicense?.substringBefore(";")?.let {`
`504`	`508`	`licenseExpressionParser.parse(it)`
`505`	`509`	`}`