feat: support EC Code and get error detail from header (#1317)

* fix: 403 error details not returned * fix: 403 error details not returned * fix: 403 error details not returned * fix: 403 error details not returned * fix: 403 error details not returned * fix: error detail from header * fix: error detail from header * fix: error detail from header * fix: error detail from header * fix: error detail from header * fix: error detail from header --------- Co-authored-by: csg01123119 <csg01123119@alibaba-inc.com>
ali-sdk · Jul 30, 2024 · 5a95778 · 5a95778
1 parent c29e15a
commit 5a95778
Show file tree

Hide file tree

Showing 5 changed files with 20,210 additions and 43 deletions.
diff --git a/lib/browser/client.js b/lib/browser/client.js
@@ -392,6 +392,32 @@ proto.parseXML = function parseXMLThunk(str) {
 
 proto.requestError = async function requestError(result) {
   let err = null;
+  const setError = async message => {
+    let info;
+    try {
+      info = (await this.parseXML(message)) || {};
+    } catch (error) {
+      this.debug(message, 'error');
+      error.message += `\nraw xml: ${message}`;
+      error.status = result.status;
+      error.requestId = result.headers['x-oss-request-id'];
+      return error;
+    }
+
+    let msg = info.Message || `unknow request error, status: ${result.status}`;
+    if (info.Condition) {
+      msg += ` (condition: ${info.Condition})`;
+    }
+    err = new Error(msg);
+    err.name = info.Code ? `${info.Code}Error` : 'UnknownError';
+    err.status = result.status;
+    err.code = info.Code;
+    err.ecCode = info.EC;
+    err.requestId = info.RequestId;
+    err.hostId = info.HostId;
+    err.serverTime = info.ServerTime;
+  };
+
   if (!result.data || !result.data.length) {
     if (result.status === -1 || result.status === -2) {
       // -1 is net error , -2 is timeout
@@ -415,6 +441,12 @@ proto.requestError = async function requestError(result) {
         err = new Error(`Unknow error, status: ${result.status}`);
         err.name = 'UnknownError';
         err.status = result.status;
+        err.res = result;
+        const ossErr = result.headers['x-oss-err'];
+        if (ossErr) {
+          const message = atob(ossErr);
+          await setError(message);
+        }
       }
       err.requestId = result.headers['x-oss-request-id'];
       err.host = '';
@@ -423,28 +455,7 @@ proto.requestError = async function requestError(result) {
     const message = String(result.data);
     this.debug('request response error data: %s', message, 'error');
 
-    let info;
-    try {
-      info = (await this.parseXML(message)) || {};
-    } catch (error) {
-      this.debug(message, 'error');
-      error.message += `\nraw xml: ${message}`;
-      error.status = result.status;
-      error.requestId = result.headers['x-oss-request-id'];
-      return error;
-    }
-
-    let msg = info.Message || `unknow request error, status: ${result.status}`;
-    if (info.Condition) {
-      msg += ` (condition: ${info.Condition})`;
-    }
-    err = new Error(msg);
-    err.name = info.Code ? `${info.Code}Error` : 'UnknownError';
-    err.status = result.status;
-    err.code = info.Code;
-    err.requestId = info.RequestId;
-    err.hostId = info.HostId;
-    err.serverTime = info.ServerTime;
+    await setError(message);
   }
 
   this.debug('generate error %j', err, 'error');

diff --git a/lib/client.js b/lib/client.js
@@ -356,6 +356,32 @@ proto.parseXML = function parseXMLThunk(str) {
 
 proto.requestError = async function requestError(result) {
   let err = null;
+
+  const setError = async message => {
+    let info;
+    try {
+      info = (await this.parseXML(message)) || {};
+    } catch (error) {
+      debug(message);
+      error.message += `\nraw xml: ${message}`;
+      error.status = result.status;
+      error.requestId = result.headers['x-oss-request-id'];
+      return error;
+    }
+
+    let msg = info.Message || `unknow request error, status: ${result.status}`;
+    if (info.Condition) {
+      msg += ` (condition: ${info.Condition})`;
+    }
+    err = new Error(msg);
+    err.name = info.Code ? `${info.Code}Error` : 'UnknownError';
+    err.status = result.status;
+    err.code = info.Code;
+    err.requestId = info.RequestId;
+    err.ecCode = info.EC;
+    err.hostId = info.HostId;
+  };
+
   if (result.name === 'ResponseTimeoutError') {
     err = new Error(result.message);
     err.name = result.name;
@@ -382,6 +408,12 @@ proto.requestError = async function requestError(result) {
         err = new Error(`Unknow error, status: ${result.status}`);
         err.name = 'UnknownError';
         err.status = result.status;
+        err.res = result;
+        const ossErr = result.headers['x-oss-err'];
+        if (ossErr) {
+          const message = Buffer.from(ossErr, 'base64').toString('utf8');
+          await setError(message);
+        }
       }
       err.requestId = result.headers['x-oss-request-id'];
       err.host = '';
@@ -390,27 +422,7 @@ proto.requestError = async function requestError(result) {
     const message = String(result.data);
     debug('request response error data: %s', message);
 
-    let info;
-    try {
-      info = (await this.parseXML(message)) || {};
-    } catch (error) {
-      debug(message);
-      error.message += `\nraw xml: ${message}`;
-      error.status = result.status;
-      error.requestId = result.headers['x-oss-request-id'];
-      return error;
-    }
-
-    let msg = info.Message || `unknow request error, status: ${result.status}`;
-    if (info.Condition) {
-      msg += ` (condition: ${info.Condition})`;
-    }
-    err = new Error(msg);
-    err.name = info.Code ? `${info.Code}Error` : 'UnknownError';
-    err.status = result.status;
-    err.code = info.Code;
-    err.requestId = info.RequestId;
-    err.hostId = info.HostId;
+    await setError(message);
   }
 
   debug('generate error %j', err);