Skip to content

Commit 6b37ad5

Browse files
committed
fix(cos): optimize error message
1 parent ca1c631 commit 6b37ad5

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

__tests__/cos.test.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,8 +86,9 @@ describe('Cos', () => {
8686
expect(
8787
convertCosError({
8888
message: 'message',
89+
requestId: '123',
8990
}).message,
90-
).toBe('message');
91+
).toBe('message (reqId: 123)');
9192

9293
expect(
9394
convertCosError({
@@ -99,9 +100,10 @@ describe('Cos', () => {
99100
convertCosError({
100101
error: {
101102
Message: 'message',
103+
RequestId: '123',
102104
},
103105
}).message,
104-
).toBe('message');
106+
).toBe('message (reqId: 123)');
105107
});
106108

107109
test('[cos] should deploy cos', async () => {

src/modules/cos/index.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,9 @@ export function convertCosError(err: CosError) {
6363
}
6464
return {
6565
code: err?.error?.Code ?? err.code!,
66-
message: err?.error?.Message ?? err.message!,
66+
message: err?.error?.Message
67+
? `${err?.error?.Message} (reqId: ${err.error.RequestId})`
68+
: `${err.message!} (reqId: ${err.requestId!})`,
6769
stack: err?.stack ?? err?.error?.Stack!,
6870
reqId: err?.error?.RequestId ?? err.requestId!,
6971
};

0 commit comments

Comments
 (0)