[TAP5-1691] AssetPathConstructorImpl should URL-encode the application version

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 5.3.1, 5.3, 5.2
• Fix Version/s: 5.4
• Component/s: tapestry-core
• Labels: patch
• Assignee: Unassigned
• Reporter: Jochen Kemnade
• Created: 13/Oct/11 09:46
• Updated: 09/Apr/13 06:25
• Resolved: 09/Apr/13 06:25

Description

When I deploy snapshot versions of T5 webapps, I usually extend the maven/gradle version number by an ISO timestamp. Therefore, my version number always contains a '+', ("...+0[12]00"). The asset urls generated by AssetPathConstructorImpl don't encode that plus using URLEncoder but ContextPathEncoderImpl.decodePath tries to decode the version string via URLEncoder.decode() which fails:

java.lang.IllegalArgumentException: Input string '3.10.0-SNAPSHOT20111010T153002+0200' is not valid; the character '+' at position 31 is not valid.