Uploaded image for project: 'Tapestry 5'
  1. Tapestry 5
  2. TAP5-1691

AssetPathConstructorImpl should URL-encode the application version

    Details

    • 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:

      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.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              jkemnade Jochen Kemnade
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: