Skip to content

Commit 40ce6b5

Browse files
Merge pull request #1790 from henrybarreto/feat/improve-test-proxy-error
feat: enhance logging for proxy testing errors
2 parents a8343a8 + ab9e0ed commit 40ce6b5

File tree

1 file changed

+12
-6
lines changed

1 file changed

+12
-6
lines changed

src/api/controllers/proxy.controller.ts

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -53,15 +53,21 @@ export class ProxyController {
5353
httpsAgent: makeProxyAgent(proxy),
5454
});
5555

56-
return response?.data !== serverIp?.data;
56+
const result = response?.data !== serverIp?.data;
57+
if (result) {
58+
logger.info('testProxy: proxy connection successful');
59+
} else {
60+
logger.warn("testProxy: proxy connection doesn't change the origin IP");
61+
}
62+
63+
return result;
5764
} catch (error) {
58-
if (axios.isAxiosError(error) && error.response?.data) {
59-
logger.error('testProxy error: ' + error.response.data);
60-
} else if (axios.isAxiosError(error)) {
61-
logger.error('testProxy error: ');
65+
if (axios.isAxiosError(error)) {
66+
logger.error('testProxy error: axios error: ' + error.message);
6267
} else {
63-
logger.error('testProxy error: ');
68+
logger.error('testProxy error: unexpected error: ' + error);
6469
}
70+
6571
return false;
6672
}
6773
}

0 commit comments

Comments
 (0)