[PR #131] [MERGED] Logging #225

New Issue

2026-02-17T17:06:28-05:00

yindo commented

2026-02-17 17:06:28 -05:00

📋 Pull Request Information

Original PR: https://github.com/Drop-OSS/drop/pull/131
Author: @Huskydog9988
Created: 7/6/2025
Status: ✅ Merged
Merged: 7/8/2025
Merged by: @DecDuck

Base: develop ← Head: logging

📝 Commits (6)

dd9a32d ci: pull version from package.json on build
e959e11 fix: implicit any type
b5048ad feat: inital support for logger
9d4aba3 style: fix lint
65a214e feat: move more logging over to pino
e0822c4 fix: logging around company importing

📊 Changes

33 files changed (+485 additions, -175 deletions)

View changed files

📝 components/GameEditor/Metadata.vue (+1 -1)
📝 components/GamePanel.vue (+3 -1)
📝 components/NewsArticleCreateButton.vue (+1 -1)
📝 components/OptionWrapper.vue (+4 -4)
📝 components/UploadFileDialog.vue (+1 -1)
📝 composables/task.ts (+3 -2)
📝 nuxt.config.ts (+54 -11)
📝 package.json (+3 -1)
📝 plugins/error-handler.ts (+3 -1)
📝 server/api/v1/auth/signin/simple.post.ts (+2 -1)
📝 server/api/v1/notifications/ws.get.ts (+2 -1)
📝 server/internal/acls/index.ts (+0 -3)
📝 server/internal/auth/index.ts (+4 -3)
📝 server/internal/auth/oidc/index.ts (+2 -1)
📝 server/internal/library/index.ts (+4 -4)
➕ server/internal/logging/index.ts (+12 -0)
📝 server/internal/metadata/giantbomb.ts (+18 -10)
📝 server/internal/metadata/igdb.ts (+24 -11)
📝 server/internal/metadata/index.ts (+7 -9)
📝 server/internal/metadata/manual.ts (+2 -2)

...and 13 more files

📄 Description

Very basic outline of server side logging. This switches everything over to pino for logging, including tasks. In future, this will allow for rotating logs, or whatever else would want to be done with the logs.

_{🔄 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/Drop-OSS/drop/pull/131 **Author:** [@Huskydog9988](https://github.com/Huskydog9988) **Created:** 7/6/2025 **Status:** ✅ Merged **Merged:** 7/8/2025 **Merged by:** [@DecDuck](https://github.com/DecDuck) **Base:** `develop` ← **Head:** `logging` --- ### 📝 Commits (6) - [`dd9a32d`](https://github.com/Drop-OSS/drop/commit/dd9a32df6c710e4523c01041cc7bc6bc191435ee) ci: pull version from package.json on build - [`e959e11`](https://github.com/Drop-OSS/drop/commit/e959e113d094c7ba666c5f65d9a82da7b4b15606) fix: implicit any type - [`b5048ad`](https://github.com/Drop-OSS/drop/commit/b5048add55b4e8a0f7ce91c92bc298a81d9404d5) feat: inital support for logger - [`9d4aba3`](https://github.com/Drop-OSS/drop/commit/9d4aba3af77425a6cf04d2b90e66fa64adf0b615) style: fix lint - [`65a214e`](https://github.com/Drop-OSS/drop/commit/65a214e82ec7e612844d0bf5c515480f4b5b9982) feat: move more logging over to pino - [`e0822c4`](https://github.com/Drop-OSS/drop/commit/e0822c4f782ecb18c4c716ffd952eb031e7bf899) fix: logging around company importing ### 📊 Changes **33 files changed** (+485 additions, -175 deletions) <details> <summary>View changed files</summary> 📝 `components/GameEditor/Metadata.vue` (+1 -1) 📝 `components/GamePanel.vue` (+3 -1) 📝 `components/NewsArticleCreateButton.vue` (+1 -1) 📝 `components/OptionWrapper.vue` (+4 -4) 📝 `components/UploadFileDialog.vue` (+1 -1) 📝 `composables/task.ts` (+3 -2) 📝 `nuxt.config.ts` (+54 -11) 📝 `package.json` (+3 -1) 📝 `plugins/error-handler.ts` (+3 -1) 📝 `server/api/v1/auth/signin/simple.post.ts` (+2 -1) 📝 `server/api/v1/notifications/ws.get.ts` (+2 -1) 📝 `server/internal/acls/index.ts` (+0 -3) 📝 `server/internal/auth/index.ts` (+4 -3) 📝 `server/internal/auth/oidc/index.ts` (+2 -1) 📝 `server/internal/library/index.ts` (+4 -4) ➕ `server/internal/logging/index.ts` (+12 -0) 📝 `server/internal/metadata/giantbomb.ts` (+18 -10) 📝 `server/internal/metadata/igdb.ts` (+24 -11) 📝 `server/internal/metadata/index.ts` (+7 -9) 📝 `server/internal/metadata/manual.ts` (+2 -2) _...and 13 more files_ </details> ### 📄 Description Very basic outline of server side logging. This switches everything over to pino for logging, including tasks. In future, this will allow for rotating logs, or whatever else would want to be done with the logs. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

yindo added the pull-request label 2026-02-17 17:06:28 -05:00

yindo closed this issue

2026-02-17 17:06:28 -05:00

yindo referenced this issue

2026-06-05 14:21:30 -04:00

[GH-ISSUE #225] Depots and installscripts #113

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Drop-OSS/drop#225