first pass at a preformant case insensitive query. mongo only so far.

Author: acinader

src/Adapters/Storage/Mongo/MongoCollection.js

@@ -13,7 +13,10 @@ export default class MongoCollection {
   // none, then build the geoindex.
   // This could be improved a lot but it's not clear if that's a good
   // idea. Or even if this behavior is a good idea.
-  find(query, { skip, limit, sort, keys, maxTimeMS, readPreference } = {}) {
+  find(
+    query,
+    { skip, limit, sort, keys, maxTimeMS, readPreference, insensitive } = {}
+  ) {
     // Support for Full Text Search - $text
     if (keys && keys.$score) {
       delete keys.$score;
@@ -26,6 +29,7 @@ export default class MongoCollection {
       keys,
       maxTimeMS,
       readPreference,
+      insensitive,
     }).catch(error => {
       // Check for "no geoindex" error
       if (
@@ -54,13 +58,17 @@ export default class MongoCollection {
               keys,
               maxTimeMS,
               readPreference,
+              insensitive,
             })
           )
       );
     });
   }
 
-  _rawFind(query, { skip, limit, sort, keys, maxTimeMS, readPreference } = {}) {
+  _rawFind(
+    query,
+    { skip, limit, sort, keys, maxTimeMS, readPreference, insensitive } = {}
+  ) {
     let findOperation = this._mongoCollection.find(query, {
       skip,
       limit,
@@ -72,6 +80,10 @@ export default class MongoCollection {
       findOperation = findOperation.project(keys);
     }
 
+    if (insensitive) {
+      findOperation = findOperation.collation({ locale: 'en_US', strength: 2 });
+    }
+
     if (maxTimeMS) {
       findOperation = findOperation.maxTimeMS(maxTimeMS);
     }

src/Adapters/Storage/Mongo/MongoStorageAdapter.js

@@ -592,7 +592,7 @@ export class MongoStorageAdapter implements StorageAdapter {
     className: string,
     schema: SchemaType,
     query: QueryType,
-    { skip, limit, sort, keys, readPreference }: QueryOptions
+    { skip, limit, sort, keys, readPreference, insensitive }: QueryOptions
   ): Promise<any> {
     schema = convertParseSchemaToMongoSchema(schema);
     const mongoWhere = transformWhere(className, query, schema);
@@ -624,6 +624,7 @@ export class MongoStorageAdapter implements StorageAdapter {
           keys: mongoKeys,
           maxTimeMS: this._maxTimeMS,
           readPreference,
+          insensitive,
         })
       )
       .then(objects =>

src/Controllers/DatabaseController.js

@@ -1210,6 +1210,7 @@ class DatabaseController {
   //   acl     restrict this operation with an ACL for the provided array
   //           of user objectIds and roles. acl: null means no user.
   //           when this field is not present, don't do anything regarding ACLs.
+  //  insensitive make string comparisons case insensitive
   // TODO: make userIds not needed here. The db adapter shouldn't know
   // anything about users, ideally. Then, improve the format of the ACL
   // arg to work like the others.
@@ -1227,6 +1228,7 @@ class DatabaseController {
       distinct,
       pipeline,
       readPreference,
+      insensitive = false,
     }: any = {},
     auth: any = {},
     validSchemaController: SchemaController.SchemaController
@@ -1271,7 +1273,14 @@ class DatabaseController {
               sort.updatedAt = sort._updated_at;
               delete sort._updated_at;
             }
-            const queryOptions = { skip, limit, sort, keys, readPreference };
+            const queryOptions = {
+              skip,
+              limit,
+              sort,
+              keys,
+              readPreference,
+              insensitive,
+            };
             Object.keys(sort).forEach(fieldName => {
               if (fieldName.match(/^authData\.([a-zA-Z0-9_]+)\.id$/)) {
                 throw new Parse.Error(

src/RestWrite.js

@@ -642,11 +642,11 @@ RestWrite.prototype._validateUserName = function() {
     .find(
       this.className,
       {
-        username: { $regex: `^${this.data.username}$`, $options: 'i' },
+        username: this.data.username,
         objectId: { $ne: this.objectId() },
         _auth_data_anonymous: null,
       },
-      { limit: 1 },
+      { limit: 1, insensitive: true },
       {},
       this.validSchemaController
     )
@@ -691,10 +691,10 @@ RestWrite.prototype._validateEmail = function() {
     .find(
       this.className,
       {
-        email: { $regex: `^${this.data.email}$`, $options: 'i' },
+        email: this.data.email,
         objectId: { $ne: this.objectId() },
       },
-      { limit: 1 },
+      { limit: 1, insensitive: true },
       {},
       this.validSchemaController
     )