Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-19123

Username/login is case insensitive on mysql, but case sensitive on postgres and oracle

    Details

      Description

      Step to reproduce :

      #1 Log in as admin in admin interface
      #2 Create new admin :
           login : TestUser1
           pw : testuser1
      #3 Log out
      #4 Try to log in to admin  interface as :
           login : TestUser1
           pw : testuser1
      #5 Log out
      #6 Try to log in to admin  interface as :
           login : testuser1
           pw : testuser1
      

      Step #6 will succeed on mysql, while fail on postgres and oracle

        Activity

        Hide
        Vidar Langseid added a comment -

        Created tc-753:Test that username is case insensitive

        Show
        Vidar Langseid added a comment - Created tc-753:Test that username is case insensitive
        Hide
        André Rømcke added a comment -

        This is also valid for Public API, in 5.2 new api's for loadUserByLogin and loadUserByEmail has been added, and both had to be documented as being case insensitive or sensitive depending on backend.

        The same is probably true for API's for loadByIdentifier and loadByRemoteId and relative criterions.

        Show
        André Rømcke added a comment - This is also valid for Public API, in 5.2 new api's for loadUserByLogin and loadUserByEmail has been added, and both had to be documented as being case insensitive or sensitive depending on backend. The same is probably true for API's for loadByIdentifier and loadByRemoteId and relative criterions.
        Show
        André Rømcke added a comment - Merged to master: https://github.com/ezsystems/ezpublish-kernel/commit/33d993211c5ef3b662dee2f5b89828883eda6bbe
        Hide
        Rui Silva (Inactive) added a comment -

        Sanity tests were executed on master with mysql, since as of currently neither postgres nor oracle are supported with ezplatform.
        No problems were found.
        Approved by QA.

        Show
        Rui Silva (Inactive) added a comment - Sanity tests were executed on master with mysql, since as of currently neither postgres nor oracle are supported with ezplatform. No problems were found. Approved by QA.
        Hide
        André Rømcke added a comment -

        Additional fix for upgrade script done: https://github.com/ezsystems/ezpublish-kernel/commit/dfd46db3a5e9ae40a492e6b58c03cb30b8168443, this will be QA'ed with upgrade instructions.

        Show
        André Rømcke added a comment - Additional fix for upgrade script done: https://github.com/ezsystems/ezpublish-kernel/commit/dfd46db3a5e9ae40a492e6b58c03cb30b8168443 , this will be QA'ed with upgrade instructions.

          People

          • Assignee:
            Unassigned
            Reporter:
            Vidar Langseid
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: