Skip to content

Commit 2cf3a33

Browse files
committed
Improve error handling
1 parent e10d014 commit 2cf3a33

File tree

2 files changed

+23
-6
lines changed

2 files changed

+23
-6
lines changed

build/index.js

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,11 +48,20 @@ server.tool("depscore", "Get the dependency score of a package with the `depscor
4848
body: JSON.stringify({ components: [{ purl }] })
4949
});
5050
const responseText = await response.text();
51-
if (response.status !== 200 || !responseText.trim()) {
52-
const errorMsg = `Error processing ${purl}`;
51+
if (response.status !== 200) {
52+
const errorMsg = `Error processing ${purl}: [${response.status}] ${responseText}`;
53+
console.error(errorMsg);
5354
return {
5455
content: [{ type: "text", text: errorMsg }],
55-
isError: true
56+
isError: false
57+
};
58+
}
59+
else if (!responseText.trim()) {
60+
const errorMsg = `${purl} was not found.`;
61+
console.error(errorMsg);
62+
return {
63+
content: [{ type: "text", text: errorMsg }],
64+
isError: false
5665
};
5766
}
5867
try {

src/index.ts

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,11 +60,19 @@ server.tool(
6060

6161
const responseText = await response.text();
6262

63-
if (response.status !== 200 || !responseText.trim()) {
64-
const errorMsg = `Error processing ${purl}`;
63+
if (response.status !== 200) {
64+
const errorMsg = `Error processing ${purl}: [${response.status}] ${responseText}`;
65+
console.error(errorMsg);
6566
return {
6667
content: [{ type: "text", text: errorMsg }],
67-
isError: true
68+
isError: false
69+
};
70+
} else if (!responseText.trim()) {
71+
const errorMsg = `${purl} was not found.`;
72+
console.error(errorMsg);
73+
return {
74+
content: [{ type: "text", text: errorMsg }],
75+
isError: false
6876
};
6977
}
7078

0 commit comments

Comments
 (0)