Derby
  1. Derby
  2. DERBY-5657

Message XCY05 for NATIVE authentication is (too?) complex

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 10.9.1.0
    • Fix Version/s: 10.9.1.0
    • Component/s: Services
    • Labels:
      None

      Description

      The error message XCY05 is rather complex:
      Invalid change of the derby.authentication.provider property. Once set to NATIVE authentication, this property cannot be changed. NATIVE::LOCAL is the only NATIVE value accepted by derby.authentication.provider. This property cannot be set to NATIVE::LOCAL unless credentials for the database owner have been stored in the database using the syscs_util.syscs_create_user procedure.

      I'd be in favor of simplifying this error message if possible, especially raising a more specific message for the error condition when the DBO hasn't been added as a user.
      Further, it was not clear to me that the information about derby.authentication.provider applies when setting it as a database property only.
      Is this message trying to describe three problem scenarios?
      a) Turning off NATIVE authentication is forbidden.
      b) Specifying an invalid value for derby.authentication.provider when set as a database property.
      c) Missing credentials for the DBO.

        Issue Links

          Activity

          Hide
          Kristian Waagan added a comment -

          Thanks, Rick.
          Closing issue.

          Show
          Kristian Waagan added a comment - Thanks, Rick. Closing issue.
          Hide
          Rick Hillegas added a comment -

          Attaching derby-5657-01-aa-splitMessage.diff. This patch splits the complex message into 3 simpler messages. Committed at subversion revision 1301064.

          Show
          Rick Hillegas added a comment - Attaching derby-5657-01-aa-splitMessage.diff. This patch splits the complex message into 3 simpler messages. Committed at subversion revision 1301064.
          Hide
          Rick Hillegas added a comment -

          Thanks for raising this issue, Kristian. I agree that splitting this message into 3 separate messages would be an improvement. Thanks.

          Show
          Rick Hillegas added a comment - Thanks for raising this issue, Kristian. I agree that splitting this message into 3 separate messages would be an improvement. Thanks.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development