15five / scim2-filter-parser

A customizable parser/transpiler for SCIM2.0 filters
Other
23 stars 8 forks source link

userName attribute should be case-insensitive, per the RFC #30

Closed logston closed 2 years ago

logston commented 2 years ago

From https://github.com/15five/django-scim2/issues/76

See https://datatracker.ietf.org/doc/html/rfc7643#section-4.1.1: (userName) This attribute is REQUIRED and is case insensitive. Currently this case-insensitive behavior is not implemented and the filter lookups are case-sensitive.

logston commented 2 years ago

To tackle this problem, we need if there is a db agnostic means of comparing two strings in a case insensitive way. For example, do all databases support UPPER (ie. UPPER(email) =UPPER(test@example.com)`)?

andersk commented 2 years ago

UPPER is part of the SQL standard, and supported by at least MySQL, PostgreSQL, and SQLite.

logston commented 2 years ago

Pushed another set of changes.