Merge pull request axios#930 from luciy/master

nickuraltsev · web-flow · commit 3579da4cb0eb · 2017-05-31T22:29:57.000-07:00
Convert the method parameter to lowercase
diff --git a/lib/core/Axios.js b/lib/core/Axios.js
@@ -35,6 +35,7 @@ Axios.prototype.request = function request(config) {
   }
 
   config = utils.merge(defaults, this.defaults, { method: 'get' }, config);
+  config.method = config.method.toLowerCase();
 
   // Support baseURL config
   if (config.baseURL && !isAbsoluteURL(config.url)) {
diff --git a/test/specs/requests.spec.js b/test/specs/requests.spec.js
@@ -17,6 +17,22 @@ describe('requests', function () {
     });
   });
 
+  it('should treat method value as lowercase string', function (done) {
+    axios({
+      url: '/foo',
+      method: 'POST'
+    }).then(function (response) {
+      expect(response.config.method).toBe('post');
+      done();
+    });
+
+    getAjaxRequest().then(function (request) {
+      request.respondWith({
+        status: 200
+      });
+    });
+  });
+
   it('should allow string arg as url, and config arg', function (done) {
     axios.post('/foo');
 

Original file line number	Diff line number	Diff line change
`@@ -35,6 +35,7 @@ Axios.prototype.request = function request(config) {`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`config = utils.merge(defaults, this.defaults, { method: 'get' }, config);`
	`38`	`+ config.method = config.method.toLowerCase();`
`38`	`39`
`39`	`40`	`// Support baseURL config`
`40`	`41`	`if (config.baseURL && !isAbsoluteURL(config.url)) {`