fix: convert to batch API when URI might be too long (#422)

Author: leeyeh

src/request.js

@@ -321,6 +321,16 @@ const AVRequest = (route, className, objectId, method, dataObject = {}, authOpti
   }
   return getServerURLPromise.then(() => {
     const apiURL = createApiUrl(route, className, objectId, method, dataObject);
+    // prevent URI too long
+    if (apiURL.length > 2000 && method.toLowerCase() === 'get') {
+      const body = {
+        request: {
+          method,
+          path: apiURL,
+        },
+      };
+      return AVRequest('batch', null, null, 'POST', body, authOptions);
+    }
     return setHeaders(authOptions).then(
       headers => ajax(method, apiURL, dataObject, headers)
         .then(

test/query.js

@@ -194,6 +194,12 @@ describe('Queries', function () {
           expect(gameScore.get('test').get('foo')).to.be('bar');
         });
     });
+
+    it('containsAll with an large array should not cause URI too long', () => {
+      return new AV.Query(GameScore)
+        .containsAll('test', new Array(1000).fill('5821abe62e958a00540046d5'))
+        .find();
+    });
   });
 
   describe('Query with different condtions', function () {