Skip to content
This repository was archived by the owner on Apr 12, 2021. It is now read-only.

Commit 480194d

Browse files
committed
format statuscodeerror
1 parent 481b911 commit 480194d

4 files changed

Lines changed: 22 additions & 8 deletions

File tree

README.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,3 +17,6 @@ const log = logger({
1717

1818
log.info(`message`);
1919
```
20+
21+
## Integrated with `request` like libs
22+
Has injected formatter that leaves only response body and status code from status code error due to possible security violation.

index.js

Lines changed: 17 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -77,12 +77,23 @@ const createLogger = (opts = {}) => {
7777
.reduce((acc, value) => ({...acc, ...value}), data);
7878

7979
if (error) {
80-
patchedInfo = {
81-
...patchedInfo,
82-
...error,
83-
stack: error.stack,
84-
message: patchedInfo.message || error.message,
85-
};
80+
if (error.statusCode && error.response && error.request) {
81+
// format http status code error due to security violation that headers w/ access token may be logged
82+
patchedInfo = {
83+
...patchedInfo,
84+
body: error.response.body,
85+
statusCode: error.response.statusCode,
86+
stack: error.stack,
87+
message: patchedInfo.message || error.message,
88+
};
89+
} else {
90+
patchedInfo = {
91+
...patchedInfo,
92+
...error,
93+
stack: error.stack,
94+
message: patchedInfo.message || error.message,
95+
};
96+
}
8697
}
8798

8899
return patchedInfo;

package-lock.json

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@vizydrop/logger",
3-
"version": "1.1.0",
3+
"version": "1.2.0",
44
"description": "Vizydrop predefined logger based on winston v3",
55
"main": "index.js",
66
"scripts": {},

0 commit comments

Comments
 (0)