Skip to content

User lookup - email address comparison is case sensitive #22

@ricemery

Description

@ricemery

Should email address domain comparison be case insensitive? I have run into a problem where Redmine is setup with lowercase email domain addresses, but Azure is returning mixed case domain email addresses.

The comparison code in the plugin is case sensitive.

From redmine_oauth_controller.rb:

  user = User.joins(:email_addresses)
               .where('email_addresses.address' => email, 'email_addresses.is_default' => true)
               .first_or_initialize

I am thinking that for Azure, comparison of the email address to the Redmine address should be case insensitive.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions