[PR #38] [MERGED] Add error handling and logging #76

Closed
opened 2026-09-04 00:22:01 +02:00 by Stone_Red · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/detleph/server/pull/38
Author: @stephan418
Created: 4/28/2022
Status: Merged
Merged: 4/28/2022
Merged by: @stephan418

Base: devHead: feature-error-handling


📝 Commits (6)

  • 07bdda5 Add basic error handling
  • c62ad87 Add logger using winston
  • 85b34ff Add logger to main
  • e8cfea6 Fix: Add winston; And: Add custom error handlers
  • e6bfb0d Add logger and remove unused code
  • 3a3f9d1 Add debugging code

📊 Changes

8 files changed (+182 additions, -19 deletions)

View changed files

📝 dev.sh (+1 -0)
📝 package.json (+3 -1)
📝 src/Controllers/event.controller.ts (+7 -7)
src/Middleware/debug/logger.ts (+10 -0)
src/Middleware/error/ForwardableError.ts (+16 -0)
src/Middleware/error/handler.ts (+82 -0)
src/Middleware/error/logger.ts (+41 -0)
📝 src/app.ts (+22 -11)

📄 Description

  • Default and custom error handling functionality
  • Loggers which only log in development
  • Loggers which only log unsensitive information when in production mode

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/detleph/server/pull/38 **Author:** [@stephan418](https://github.com/stephan418) **Created:** 4/28/2022 **Status:** ✅ Merged **Merged:** 4/28/2022 **Merged by:** [@stephan418](https://github.com/stephan418) **Base:** `dev` ← **Head:** `feature-error-handling` --- ### 📝 Commits (6) - [`07bdda5`](https://github.com/detleph/server/commit/07bdda5ef930037a0c6bd37d6e7332b87dbdf4d2) Add basic error handling - [`c62ad87`](https://github.com/detleph/server/commit/c62ad879463beca3d6cf89c6d9afddf506ac00ed) Add logger using winston - [`85b34ff`](https://github.com/detleph/server/commit/85b34ff66ea2204edc990408aff93782e2503174) Add logger to main - [`e8cfea6`](https://github.com/detleph/server/commit/e8cfea67cec9181ff1070426e2208e677c182e91) Fix: Add winston; And: Add custom error handlers - [`e6bfb0d`](https://github.com/detleph/server/commit/e6bfb0da95062b38506a1b44130d81e22b6a6991) Add logger and remove unused code - [`3a3f9d1`](https://github.com/detleph/server/commit/3a3f9d1fd2969949aa02b3bdba41d651f73f266f) Add debugging code ### 📊 Changes **8 files changed** (+182 additions, -19 deletions) <details> <summary>View changed files</summary> 📝 `dev.sh` (+1 -0) 📝 `package.json` (+3 -1) 📝 `src/Controllers/event.controller.ts` (+7 -7) ➕ `src/Middleware/debug/logger.ts` (+10 -0) ➕ `src/Middleware/error/ForwardableError.ts` (+16 -0) ➕ `src/Middleware/error/handler.ts` (+82 -0) ➕ `src/Middleware/error/logger.ts` (+41 -0) 📝 `src/app.ts` (+22 -11) </details> ### 📄 Description + Default and custom error handling functionality + Loggers which only log in development + Loggers which only log unsensitive information when in production mode --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Stone_Red added the pull-request label 2026-09-04 00:22:01 +02:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: detleph/server#76