Commit ceb9e21
committed
review: log { err } in failure guards so stacks survive
String interpolation dropped Error stack traces; { err } is the house
idiom (src/server.js:1127), pino serializes it with stack and console
prints it whole, and it handles non-Error throws without touching
.message — which also retires the errMessage() helper from the previous
round. All three guard sites (mountApp fail, both ws.route paths).
Suite 1024/1024; non-Error regression tests unchanged and passing.1 parent 92ffb28 commit ceb9e21
1 file changed
Lines changed: 3 additions & 12 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
60 | 60 | | |
61 | 61 | | |
62 | 62 | | |
63 | | - | |
64 | | - | |
65 | | - | |
66 | | - | |
67 | | - | |
68 | | - | |
69 | | - | |
70 | | - | |
71 | | - | |
72 | 63 | | |
73 | 64 | | |
74 | 65 | | |
| |||
196 | 187 | | |
197 | 188 | | |
198 | 189 | | |
199 | | - | |
| 190 | + | |
200 | 191 | | |
201 | 192 | | |
202 | 193 | | |
| |||
233 | 224 | | |
234 | 225 | | |
235 | 226 | | |
236 | | - | |
| 227 | + | |
237 | 228 | | |
238 | 229 | | |
239 | 230 | | |
240 | | - | |
| 231 | + | |
241 | 232 | | |
242 | 233 | | |
243 | 234 | | |
| |||
0 commit comments