Derby
  1. Derby
  2. DERBY-2754

Client connections should not fail if permission is not granted to read derby.client.traceLevel and derby.client.traceDirectory

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 10.3.1.4
    • Fix Version/s: 10.3.1.4
    • Component/s: Network Client
    • Labels:
      None
    • Urgency:
      Urgent
    • Bug behavior facts:
      Regression

      Description

      Running the 10.2 network server tests with the 10.3 server, most tests fail with:
      AccessControlException: Access denied java.util.Property derby.client.traceLevel read
      due to DERBY-1275. This can break existing applications that don't have this permission and should be a non-fatal warning if permission is not granted.

        Issue Links

          Activity

          Hide
          Kathey Marsden added a comment -

          This issue allows client connections to proceed even if permission is not granted for the new system properties.

          Show
          Kathey Marsden added a comment - This issue allows client connections to proceed even if permission is not granted for the new system properties.
          Hide
          Kathey Marsden added a comment -

          committed rev 545251

          Show
          Kathey Marsden added a comment - committed rev 545251
          Hide
          Kathey Marsden added a comment -

          Existing applications should no longer be impacted now that this is fixed.

          Show
          Kathey Marsden added a comment - Existing applications should no longer be impacted now that this is fixed.
          Hide
          Daniel John Debrunner added a comment -

          Patch looks good, though I think it would be good to add some comments explaining what is going on.
          The comment you did add just describes what the code is doing, so it doesn't help out with the why.

          Show
          Daniel John Debrunner added a comment - Patch looks good, though I think it would be good to add some comments explaining what is going on. The comment you did add just describes what the code is doing, so it doesn't help out with the why.
          Hide
          Kathey Marsden added a comment -

          Attached is a patch for this issue. I ran Suites.All, derbyall and ran derbynetclientmats with 10.2 derbyTesting.jar to verify the issue was fixed. Tests did not fail with the SecurityException after the change. Other failures with the 10.2 tests still need to be analyzed.

          Show
          Kathey Marsden added a comment - Attached is a patch for this issue. I ran Suites.All, derbyall and ran derbynetclientmats with 10.2 derbyTesting.jar to verify the issue was fixed. Tests did not fail with the SecurityException after the change. Other failures with the 10.2 tests still need to be analyzed.
          Hide
          Kathey Marsden added a comment -

          There does not seem to be any place to put a warning at this point of processing, so if permission is not granted to read the properties, they will just be ignored. This may cause some confusion, but I will add a not to the page describing the properties to help troubleshoot if the properties are set but tracing is not affected.

          Show
          Kathey Marsden added a comment - There does not seem to be any place to put a warning at this point of processing, so if permission is not granted to read the properties, they will just be ignored. This may cause some confusion, but I will add a not to the page describing the properties to help troubleshoot if the properties are set but tracing is not affected.

            People

            • Assignee:
              Kathey Marsden
              Reporter:
              Kathey Marsden
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development