Derby
  1. Derby
  2. DERBY-5897

Broken handling of paths with special characters in compatibility/VersionCombinationConfigurator

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 10.10.1.1
    • Fix Version/s: 10.10.1.1
    • Component/s: Test
    • Labels:
      None

      Description

      Some of the code handling paths in compatibility/VersionCombinationConfigurator is broken.
      The issue is that the location of a given class is given as a URL. This means the path will be URL encoded, and, for instance, spaces will be encoded as '%20'.

      This bug typically shows itself as an NPE when invoking compatibility._Suite.

        Activity

        Kristian Waagan created issue -
        Kristian Waagan made changes -
        Field Original Value New Value
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Kristian Waagan added a comment -

        Attaching patch 1a, which fixes the encoded URL issue by converting the URL to an URI and passing it to the File(URI) constructor.

        Also added a new sanity check (fail-fast) and a debug printout.

        Patch ready for review.

        Show
        Kristian Waagan added a comment - Attaching patch 1a, which fixes the encoded URL issue by converting the URL to an URI and passing it to the File(URI) constructor. Also added a new sanity check (fail-fast) and a debug printout. Patch ready for review.
        Kristian Waagan made changes -
        Attachment derby-5897-1a-url_handling.diff [ 12539996 ]
        Kristian Waagan made changes -
        Issue & fix info Patch Available [ 10102 ]
        Hide
        Kristian Waagan added a comment -

        Committed patch 1a, with some formatting modifications, to trunk with revision 1372345.

        Show
        Kristian Waagan added a comment - Committed patch 1a, with some formatting modifications, to trunk with revision 1372345.
        Kristian Waagan made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Issue & fix info Patch Available [ 10102 ]
        Fix Version/s 10.10.0.0 [ 12321550 ]
        Resolution Fixed [ 1 ]
        Gavin made changes -
        Workflow jira [ 12720195 ] Default workflow, editable Closed status [ 12801741 ]
        Hide
        Knut Anders Hatlen added a comment -

        [bulk update: close all resolved issues that haven't had any activity the last year]

        Show
        Knut Anders Hatlen added a comment - [bulk update: close all resolved issues that haven't had any activity the last year]
        Knut Anders Hatlen made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        1h 21m 1 Kristian Waagan 09/Aug/12 10:01
        In Progress In Progress Resolved Resolved
        4d 52m 1 Kristian Waagan 13/Aug/12 10:54
        Resolved Resolved Closed Closed
        458d 22h 20m 1 Knut Anders Hatlen 15/Nov/13 08:15

          People

          • Assignee:
            Kristian Waagan
            Reporter:
            Kristian Waagan
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development