allow optional HTTP POST method override for logIn user method

Author: kolohelios

src/ParseUser.js

@@ -439,7 +439,7 @@ class ParseUser extends ParseObject {
    * @return {Promise} A promise that is fulfilled with the user when
    *     the login is complete.
    */
-  logIn(options?: FullOptions): Promise<ParseUser> {
+  logIn(options?: FullOptions, usePost?: boolean): Promise<ParseUser> {
     options = options || {};
 
     const loginOptions = {};
@@ -451,7 +451,7 @@ class ParseUser extends ParseObject {
     }
 
     const controller = CoreManager.getUserController();
-    return controller.logIn(this, loginOptions);
+    return controller.logIn(this, loginOptions, usePost);
   }
 
   /**
@@ -628,11 +628,12 @@ class ParseUser extends ParseObject {
    * @param {String} username The username (or email) to log in with.
    * @param {String} password The password to log in with.
    * @param {Object} options
+   * @param {Boolean} usePost Override HTTP REST method and use POST instead of GET
    * @static
    * @return {Promise} A promise that is fulfilled with the user when
    *     the login completes.
    */
-  static logIn(username: string, password: string, options?: FullOptions) {
+  static logIn(username: string, password: string, options?: FullOptions, usePost?: boolean) {
     if (typeof username !== 'string') {
       return Promise.reject(
         new ParseError(
@@ -650,7 +651,7 @@ class ParseUser extends ParseObject {
     }
     const user = new this();
     user._finishFetch({ username: username, password: password });
-    return user.logIn(options);
+    return user.logIn(options, usePost);
   }
 
   /**
@@ -1096,15 +1097,15 @@ const DefaultController = {
     });
   },
 
-  logIn(user: ParseUser, options: RequestOptions): Promise<ParseUser> {
+  logIn(user: ParseUser, options: RequestOptions, usePost?: boolean): Promise<ParseUser> {
     const RESTController = CoreManager.getRESTController();
     const stateController = CoreManager.getObjectStateController();
     const auth = {
       username: user.get('username'),
       password: user.get('password')
     };
     return RESTController.request(
-      'GET', 'login', auth, options
+      usePost ? 'POST' : 'GET', 'login', auth, options
     ).then((response) => {
       user._migrateId(response.objectId);
       user._setExisted(true);

src/__tests__/ParseUser-test.js

@@ -229,6 +229,39 @@ describe('ParseUser', () => {
     });
   });
 
+  it('can log in as a user with POST method', (done) => {
+    ParseUser.enableUnsafeCurrentUser();
+    ParseUser._clearCache();
+    CoreManager.setRESTController({
+      request(method, path, body) {
+        expect(method).toBe('POST');
+        expect(path).toBe('login');
+        expect(body.username).toBe('username');
+        expect(body.password).toBe('password');
+
+        return Promise.resolve({
+          objectId: 'uid2',
+          username: 'username',
+          sessionToken: '123abc'
+        }, 200);
+      },
+      ajax() {}
+    });
+    ParseUser.logIn('username', 'password', {}, true).then((u) => {
+      expect(u.id).toBe('uid2');
+      expect(u.getSessionToken()).toBe('123abc');
+      expect(u.isCurrent()).toBe(true);
+      expect(u.authenticated()).toBe(true);
+      expect(ParseUser.current()).toBe(u);
+      ParseUser._clearCache();
+      const current = ParseUser.current();
+      expect(current instanceof ParseUser).toBe(true);
+      expect(current.id).toBe('uid2');
+      expect(current.authenticated()).toBe(true);
+      done();
+    });
+  });
+
   it('fail login when invalid username or password is used', (done) => {
     ParseUser.enableUnsafeCurrentUser();
     ParseUser._clearCache();