[PR #787] [MERGED] refactor: complete conversion of log calls to slog #618

New Issue

OVERLORD · 2025-10-08T00:13:59+03:00

OVERLORD commented

2025-10-08 00:13:59 +03:00

📋 Pull Request Information

Original PR: https://github.com/pocket-id/pocket-id/pull/787
Author: @ItalyPaleAle
Created: 7/27/2025
Status: ✅ Merged
Merged: 7/27/2025
Merged by: @ItalyPaleAle

Base: main ← Head: convert-logs

📝 Commits (2)

9d9f6c7 refactor: complete conversion of log calls to slog
7b03152 Review feedback

📊 Changes

22 files changed (+142 additions, -88 deletions)

View changed files

📝 backend/internal/bootstrap/bootstrap.go (+4 -1)
📝 backend/internal/bootstrap/db_bootstrap.go (+8 -9)
📝 backend/internal/bootstrap/e2etest_router_bootstrap.go (+4 -2)
📝 backend/internal/bootstrap/services_bootstrap.go (+9 -2)
📝 backend/internal/cmds/key_rotate.go (+8 -2)
📝 backend/internal/cmds/key_rotate_test.go (+4 -2)
📝 backend/internal/cmds/one_time_access_token.go (+5 -2)
📝 backend/internal/common/env_config.go (+4 -2)
📝 backend/internal/controller/e2etest_controller.go (+1 -1)
📝 backend/internal/controller/well_known_controller.go (+5 -2)
📝 backend/internal/dto/validations.go (+8 -5)
📝 backend/internal/service/app_config_service.go (+4 -5)
📝 backend/internal/service/audit_log_service.go (+6 -1)
📝 backend/internal/service/e2etest_service.go (+3 -3)
📝 backend/internal/service/geolite_service.go (+2 -1)
📝 backend/internal/service/ldap_service.go (+2 -3)
📝 backend/internal/service/oidc_service.go (+32 -23)
📝 backend/internal/service/user_service.go (+15 -8)
📝 backend/internal/service/webauthn_service.go (+7 -4)
📝 backend/internal/utils/aaguid_util.go (+5 -4)

...and 2 more files

📄 Description

Follow-up from #760

A few tiny bugs were fixed in the process :)

_{🔄 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/pocket-id/pocket-id/pull/787 **Author:** [@ItalyPaleAle](https://github.com/ItalyPaleAle) **Created:** 7/27/2025 **Status:** ✅ Merged **Merged:** 7/27/2025 **Merged by:** [@ItalyPaleAle](https://github.com/ItalyPaleAle) **Base:** `main` ← **Head:** `convert-logs` --- ### 📝 Commits (2) - [`9d9f6c7`](https://github.com/pocket-id/pocket-id/commit/9d9f6c746df243481bae5a4064aa0b11d70f5d88) refactor: complete conversion of log calls to slog - [`7b03152`](https://github.com/pocket-id/pocket-id/commit/7b03152eae7e0af70435ab24f2df509f83ffdce8) Review feedback ### 📊 Changes **22 files changed** (+142 additions, -88 deletions) <details> <summary>View changed files</summary> 📝 `backend/internal/bootstrap/bootstrap.go` (+4 -1) 📝 `backend/internal/bootstrap/db_bootstrap.go` (+8 -9) 📝 `backend/internal/bootstrap/e2etest_router_bootstrap.go` (+4 -2) 📝 `backend/internal/bootstrap/services_bootstrap.go` (+9 -2) 📝 `backend/internal/cmds/key_rotate.go` (+8 -2) 📝 `backend/internal/cmds/key_rotate_test.go` (+4 -2) 📝 `backend/internal/cmds/one_time_access_token.go` (+5 -2) 📝 `backend/internal/common/env_config.go` (+4 -2) 📝 `backend/internal/controller/e2etest_controller.go` (+1 -1) 📝 `backend/internal/controller/well_known_controller.go` (+5 -2) 📝 `backend/internal/dto/validations.go` (+8 -5) 📝 `backend/internal/service/app_config_service.go` (+4 -5) 📝 `backend/internal/service/audit_log_service.go` (+6 -1) 📝 `backend/internal/service/e2etest_service.go` (+3 -3) 📝 `backend/internal/service/geolite_service.go` (+2 -1) 📝 `backend/internal/service/ldap_service.go` (+2 -3) 📝 `backend/internal/service/oidc_service.go` (+32 -23) 📝 `backend/internal/service/user_service.go` (+15 -8) 📝 `backend/internal/service/webauthn_service.go` (+7 -4) 📝 `backend/internal/utils/aaguid_util.go` (+5 -4) _...and 2 more files_ </details> ### 📄 Description Follow-up from #760 A few tiny bugs were fixed in the process :) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2025-10-08 00:13:59 +03:00

OVERLORD closed this issue

2025-10-08 00:14:01 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/pocket-id-pocket-id-1#618