Skip to content

Commit 1386ba8

Browse files
committed
fix(file): handle failed (responsed but not OK) upload properly
1 parent a3326e1 commit 1386ba8

File tree

2 files changed

+18
-0
lines changed

2 files changed

+18
-0
lines changed

src/uploader/cos.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,11 @@ module.exports = function(uploadInfo, data, file, saveOptions = {}) {
2424
return upload(url, fileFormData, options).then(
2525
response => {
2626
debug(response.status, response.data);
27+
if (response.ok === false) {
28+
const error = new Error(response.status);
29+
error.response = response;
30+
throw error;
31+
}
2732
return file;
2833
},
2934
error => {

src/uploader/qiniu.js

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,19 @@ module.exports = function(uploadInfo, data, file, saveOptions = {}) {
2727
return upload(url, fileFormData, options).then(
2828
response => {
2929
debug(response.status, response.data);
30+
if (response.ok === false) {
31+
let message = response.status;
32+
if (response.data) {
33+
if (response.data.error) {
34+
message = response.data.error;
35+
} else {
36+
message = JSON.stringify(response.data);
37+
}
38+
}
39+
const error = new Error(message);
40+
error.response = response;
41+
throw error;
42+
}
3043
return file;
3144
},
3245
error => {

0 commit comments

Comments
 (0)