tree e05ea14adb8833207397936327f00764d2b04ebe
parent 1f215f8ca1c459e4c564f6c9b3f88d1cfa7a113a
author mgalindo-sc <161385164+mgalindo-sc@users.noreply.github.com> 1781286286 -0700
committer rules_java Copybara <noreply@google.com> 1781286306 -0700

Do not percent-encode '+' in classpath-jar manifest Class-Path entries (#358)

Add '+' to the allowed character class so it passes through literally.

The java stub template packs over-long classpaths into a classpath jar whose manifest Class-Path entries percent-encode any character outside [-_.~/a-zA-Z0-9]. Bzlmod canonical repository names put '+' into every external-repo classpath entry, so those entries now contain "%2b" sequences. '+' is a valid URI path character and needs no encoding, and the "%2b" breaks consumers that pass classpath URLs through printf-style formatting

Closes #358

COPYBARA_INTEGRATE_REVIEW=https://github.com/bazelbuild/rules_java/pull/358 from mgalindo-sc:patch-1 367ccce6a08bb163cfde47ebe44fcdf34593f463
PiperOrigin-RevId: 931225696
Change-Id: I3c4e215afc4c67e25e48485cd58d0c2fe2dcd74c