Notes

Author: drew-gross

spec/ParseUser.spec.js

@@ -2273,4 +2273,26 @@ describe('Parse.User testing', () => {
     });
   });
 
+  it('should not allow you to sign up with facebook twice (regression test for #1488)', done => {
+    const signUpPromise1 = Parse.FacebookUtils.logIn({
+      id: "8675309",
+      access_token: "jenny",
+      expiration_date: new Date().toJSON()
+    }).;st signUpPromise2 = Parse.FacebookUtils.logIn({
+      id: "8675309",
+      access_token: "jenny",
+      expiration_date: new Date().toJSON()
+    });
+    Promise.all([signUpPromise1, signUpPromise2]).then(() => {
+      new Parse.Query(Parse.User)
+      .find({ useMasterKey: true })
+      .then(allUsers => {
+        expect(allUsers.length).toEqual(1);
+        done();
+      });
+    }, e => {
+      console.log(e);
+      done();
+    })
+  });
 });

src/RestWrite.js

@@ -365,6 +365,8 @@ RestWrite.prototype.transformUser = function() {
 
   }).then(() => {
     // Check for username uniqueness
+    //TODO: use a unique index on usernames to check for uniqueness to avoid race conditions
+    //TODO: use a unique index on auth data to fix issue #1488
     if (!this.data.username) {
       if (!this.query) {
         this.data.username = cryptoUtils.randomString(25);