Derby
  1. Derby
  2. DERBY-4475

[patch] roleName isn't trimmed as expected

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 10.5.3.0
    • Fix Version/s: 10.5.3.2, 10.6.1.0
    • Component/s: JDBC
    • Labels:
      None

      Description

      (from patch)

      public static String parseRoleId(String roleName) throws StandardException
      {

      • roleName.trim();
        + roleName = roleName.trim();
        // NONE is a special case and is not allowed with its special
        // meaning in SET ROLE <value specification>. Even if there is
        // a role with case normal form "NONE", we require it to be
      1. trim.diff
        0.6 kB
        Dave Brosius
      2. derby-4475.diff
        2 kB
        Dag H. Wanvik
      3. derby-4475.stat
        0.1 kB
        Dag H. Wanvik

        Issue Links

          Activity

          Hide
          Dag H. Wanvik added a comment -

          Thanks, Dave. Uploading a patch derby-4475 which includes your patch + new test cases.
          Will commit this shortly as soon as I have run all regression tests.

          Show
          Dag H. Wanvik added a comment - Thanks, Dave. Uploading a patch derby-4475 which includes your patch + new test cases. Will commit this shortly as soon as I have run all regression tests.
          Hide
          Dag H. Wanvik added a comment -

          Regressions passed, committed as svn 891350. Thanks Dave.
          If you have a developer id, you may want to assign yourself to this issue.
          I have marked it as resolved, pending your acceptance - if you agree that the issue has been solved, please close the issue (or I can do it).

          Show
          Dag H. Wanvik added a comment - Regressions passed, committed as svn 891350. Thanks Dave. If you have a developer id, you may want to assign yourself to this issue. I have marked it as resolved, pending your acceptance - if you agree that the issue has been solved, please close the issue (or I can do it).
          Hide
          Kathey Marsden added a comment -

          Reopen for backport

          Show
          Kathey Marsden added a comment - Reopen for backport
          Hide
          Kathey Marsden added a comment -

          Resolving after backport to 10.5. Assigning to Dave Brosius who submitted the patch. Thanks Dave!

          Show
          Kathey Marsden added a comment - Resolving after backport to 10.5. Assigning to Dave Brosius who submitted the patch. Thanks Dave!
          Hide
          Knut Anders Hatlen added a comment -

          [bulk update] Close all resolved issues that haven't been updated for more than one year.

          Show
          Knut Anders Hatlen added a comment - [bulk update] Close all resolved issues that haven't been updated for more than one year.

            People

            • Assignee:
              Dave Brosius
              Reporter:
              Dave Brosius
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development