Derby
  1. Derby
  2. DERBY-5763

Reference Guide should note the new casing rules for the USERNAME arguments to the NATIVE procedures

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 10.9.1.0
    • Fix Version/s: 10.9.1.0
    • Component/s: Documentation
    • Labels:
      None
    • Urgency:
      Normal

      Description

      The work on DERBY-5762 changed the behavior of the USERNAME arguments to the NATIVE procedures. Now those arguments are case-sensitive only if you double-quote the username string. For example, the following calls affect the same user, EDWARD:

      call syscs_util.syscs_create_user( 'Edward', 'hisPassword' )
      call syscs_util.syscs_create_user( 'EdWard', 'hisPassword' )

      ...while the following call affects a different user, EdWard:

      call syscs_util.syscs_create_user( '"EdWard"', 'hisPassword' )

      The rule is this: without double quotes, the USERNAME argument is uppercased. To create camel-case or lowercase users, you need to double-quote the USERNAME.

      1. DERBY-5763.diff
        8 kB
        Kim Haase
      2. DERBY-5763.stat
        0.2 kB
        Kim Haase
      3. DERBY-5763.zip
        13 kB
        Kim Haase
      4. DERBY-5763-2.diff
        9 kB
        Kim Haase
      5. DERBY-5763-2.zip
        13 kB
        Kim Haase

        Issue Links

          Activity

          Hide
          Kim Haase added a comment -

          I hope I've fixed all the relevant topics here. I'm attaching DERBY-5763.diff, DERBY-5763.stat, and DERBY-5763.zip, with changes to the following topics:

          M src/ref/rrefnativecreateuserproc.dita
          M src/ref/rrefnativedropuserproc.dita
          M src/ref/rrefnativeresetpasswordproc.dita
          M src/devguide/cdevcsecure24458.dita
          M src/devguide/cdevcsecurenativeauth.dita

          Please let me know what changes are needed.

          Show
          Kim Haase added a comment - I hope I've fixed all the relevant topics here. I'm attaching DERBY-5763 .diff, DERBY-5763 .stat, and DERBY-5763 .zip, with changes to the following topics: M src/ref/rrefnativecreateuserproc.dita M src/ref/rrefnativedropuserproc.dita M src/ref/rrefnativeresetpasswordproc.dita M src/devguide/cdevcsecure24458.dita M src/devguide/cdevcsecurenativeauth.dita Please let me know what changes are needed.
          Hide
          Rick Hillegas added a comment -

          Thanks for the patch, Kim. Looks good. A couple comments:

          cdevcsecurenativeauth:

          o First bullet: While you're in there, I noticed something which could be clarified now that NATIVE authentication is turned on by storing the DBO's credentials. The sentence beginning "Individual databases can override" would be more accurate if we rewrote it like this:

          "If an individual database holds credentials for the database owner, then the global credentials database is only used to authenticate system-wide operations such as engine shutdown."

          o Third bullet: Similarly, I would remove this bullet altogether.

          Thanks,
          -Rick

          Show
          Rick Hillegas added a comment - Thanks for the patch, Kim. Looks good. A couple comments: cdevcsecurenativeauth: o First bullet: While you're in there, I noticed something which could be clarified now that NATIVE authentication is turned on by storing the DBO's credentials. The sentence beginning "Individual databases can override" would be more accurate if we rewrote it like this: "If an individual database holds credentials for the database owner, then the global credentials database is only used to authenticate system-wide operations such as engine shutdown." o Third bullet: Similarly, I would remove this bullet altogether. Thanks, -Rick
          Hide
          Kim Haase added a comment -

          Thanks, Rick. By the "Third bullet" do you mean the one that describes the "NATIVE::LOCAL" setting that should not be specified explicitly? If so, I also need to remove that bullet item from the derby.authentication.provider topic in the Reference Manual. Maybe we need a new JIRA for that, or else we should change the title of this issue to include other fixes to the NATIVE authentication topics. What do you think?

          Show
          Kim Haase added a comment - Thanks, Rick. By the "Third bullet" do you mean the one that describes the "NATIVE::LOCAL" setting that should not be specified explicitly? If so, I also need to remove that bullet item from the derby.authentication.provider topic in the Reference Manual. Maybe we need a new JIRA for that, or else we should change the title of this issue to include other fixes to the NATIVE authentication topics. What do you think?
          Hide
          Rick Hillegas added a comment -

          Hi Kim,

          Yes, that's the bullet I meant. Another JIRA sounds like a good idea. Thanks.

          Show
          Rick Hillegas added a comment - Hi Kim, Yes, that's the bullet I meant. Another JIRA sounds like a good idea. Thanks.
          Hide
          Kim Haase added a comment -

          Thanks, Rick. Here's a second patch with the change to the first bullet item. I'll file a second JIRA for the removal of mentions of NATIVE::LOCAL as a setting for derby.authentication.provider.

          Show
          Kim Haase added a comment - Thanks, Rick. Here's a second patch with the change to the first bullet item. I'll file a second JIRA for the removal of mentions of NATIVE::LOCAL as a setting for derby.authentication.provider.
          Hide
          Rick Hillegas added a comment -

          Thanks for the second patch and the additional JIRA, Kim. +1

          Show
          Rick Hillegas added a comment - Thanks for the second patch and the additional JIRA, Kim. +1
          Hide
          Kim Haase added a comment -

          Thanks, Rick!

          Committed patch DERBY-5763-2.diff to documentation trunk at revision 1339604.

          Show
          Kim Haase added a comment - Thanks, Rick! Committed patch DERBY-5763 -2.diff to documentation trunk at revision 1339604.
          Hide
          Kim Haase added a comment -

          Changes have appeared in Latest Alpha Manuals.

          Show
          Kim Haase added a comment - Changes have appeared in Latest Alpha Manuals.

            People

            • Assignee:
              Kim Haase
              Reporter:
              Rick Hillegas
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development