mirror of
https://github.com/tenrok/axios.git
synced 2026-06-17 19:21:29 +03:00
Merge pull request #930 from luciy/master
Convert the method parameter to lowercase
This commit is contained in:
@@ -35,6 +35,7 @@ Axios.prototype.request = function request(config) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
config = utils.merge(defaults, this.defaults, { method: 'get' }, config);
|
config = utils.merge(defaults, this.defaults, { method: 'get' }, config);
|
||||||
|
config.method = config.method.toLowerCase();
|
||||||
|
|
||||||
// Support baseURL config
|
// Support baseURL config
|
||||||
if (config.baseURL && !isAbsoluteURL(config.url)) {
|
if (config.baseURL && !isAbsoluteURL(config.url)) {
|
||||||
|
|||||||
@@ -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) {
|
it('should allow string arg as url, and config arg', function (done) {
|
||||||
axios.post('/foo');
|
axios.post('/foo');
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user