[PR #90] [MERGED] refactor: organize code #165

New Issue

adam · 2025-12-29T09:23:44+01:00

adam commented

2025-12-29 09:23:44 +01:00

📋 Pull Request Information

Original PR: https://github.com/yusing/godoxy/pull/90
Author: @yusing
Created: 4/17/2025
Status: ✅ Merged
Merged: 4/17/2025
Merged by: @yusing

Base: main ← Head: refactor/organize-code

📝 Commits (10+)

cfd1d8f fix: improved sync.Pool handling
c7e0dcb refactor: ping if-flow and remove timeout
0866feb refactor: enhance favicon fetching with context support and improve cache management
c80a557 fix: log formatting
9af42a9 feat(pool): add checkExists method to debug build to detect unexpected behavior
7f4b04e chore: cont. 0866feb
1fe21b8 refactor: unify route handling by consolidating route query methods with Pool
41bef09 chore: uncomment icon list cache code
78bed43 refactor: update task management code
0a13bcb refactor: enhance idlewatcher logging and exit handling

📊 Changes

47 files changed (+512 additions, -449 deletions)

View changed files

📝 agent/pkg/agent/agents.go (+2 -2)
📝 cmd/main.go (+2 -2)
📝 internal/api/v1/favicon/favicon.go (+3 -3)
📝 internal/api/v1/health.go (+2 -2)
📝 internal/api/v1/list.go (+6 -6)
📝 internal/docker/client.go (+1 -1)
📝 internal/entrypoint/entrypoint.go (+6 -7)
📝 internal/entrypoint/entrypoint_test.go (+18 -10)
📝 internal/gperr/log.go (+2 -2)
📝 internal/gperr/multiline_test.go (+0 -9)
📝 internal/homepage/favicon.go (+59 -32)
📝 internal/homepage/icon_cache.go (+49 -26)
📝 internal/homepage/list-icons.go (+9 -9)
📝 internal/homepage/route.go (+3 -1)
📝 internal/idlewatcher/watcher.go (+11 -7)
📝 internal/metrics/period/poller.go (+1 -1)
📝 internal/metrics/uptime/uptime.go (+4 -5)
📝 internal/net/gphttp/accesslog/access_logger.go (+8 -10)
📝 internal/net/gphttp/accesslog/access_logger_test.go (+23 -1)
📝 internal/net/gphttp/accesslog/formatter.go (+30 -51)

...and 27 more files

📄 Description

No description provided

_{🔄 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/yusing/godoxy/pull/90 **Author:** [@yusing](https://github.com/yusing) **Created:** 4/17/2025 **Status:** ✅ Merged **Merged:** 4/17/2025 **Merged by:** [@yusing](https://github.com/yusing) **Base:** `main` ← **Head:** `refactor/organize-code` --- ### 📝 Commits (10+) - [`cfd1d8f`](https://github.com/yusing/godoxy/commit/cfd1d8fff0f0443028bbebca06cf0fcff39034f4) fix: improved sync.Pool handling - [`c7e0dcb`](https://github.com/yusing/godoxy/commit/c7e0dcbfd8e3dc3c96c0a2c58d4bce1af8608136) refactor: ping if-flow and remove timeout - [`0866feb`](https://github.com/yusing/godoxy/commit/0866feb81b57604449d2ca1b7b63f52b55af6119) refactor: enhance favicon fetching with context support and improve cache management - [`c80a557`](https://github.com/yusing/godoxy/commit/c80a557a427afab4518f567ccf45d274743afbf5) fix: log formatting - [`9af42a9`](https://github.com/yusing/godoxy/commit/9af42a9439141ee18056fefa6cf108653209860f) feat(pool): add checkExists method to debug build to detect unexpected behavior - [`7f4b04e`](https://github.com/yusing/godoxy/commit/7f4b04efb76ea6e8e404078853ae49184be96a53) chore: cont. 0866feb - [`1fe21b8`](https://github.com/yusing/godoxy/commit/1fe21b84eb317aa6b328c6a7cb80d3aafbbabf8c) refactor: unify route handling by consolidating route query methods with Pool - [`41bef09`](https://github.com/yusing/godoxy/commit/41bef09be298225f335b8606ab7a47a350198ab8) chore: uncomment icon list cache code - [`78bed43`](https://github.com/yusing/godoxy/commit/78bed431994fc0f93290fe3d9efcd4dcb3b85bb3) refactor: update task management code - [`0a13bcb`](https://github.com/yusing/godoxy/commit/0a13bcb8a3711083f8c279caa4e5c4ce3dfb549b) refactor: enhance idlewatcher logging and exit handling ### 📊 Changes **47 files changed** (+512 additions, -449 deletions) <details> <summary>View changed files</summary> 📝 `agent/pkg/agent/agents.go` (+2 -2) 📝 `cmd/main.go` (+2 -2) 📝 `internal/api/v1/favicon/favicon.go` (+3 -3) 📝 `internal/api/v1/health.go` (+2 -2) 📝 `internal/api/v1/list.go` (+6 -6) 📝 `internal/docker/client.go` (+1 -1) 📝 `internal/entrypoint/entrypoint.go` (+6 -7) 📝 `internal/entrypoint/entrypoint_test.go` (+18 -10) 📝 `internal/gperr/log.go` (+2 -2) 📝 `internal/gperr/multiline_test.go` (+0 -9) 📝 `internal/homepage/favicon.go` (+59 -32) 📝 `internal/homepage/icon_cache.go` (+49 -26) 📝 `internal/homepage/list-icons.go` (+9 -9) 📝 `internal/homepage/route.go` (+3 -1) 📝 `internal/idlewatcher/watcher.go` (+11 -7) 📝 `internal/metrics/period/poller.go` (+1 -1) 📝 `internal/metrics/uptime/uptime.go` (+4 -5) 📝 `internal/net/gphttp/accesslog/access_logger.go` (+8 -10) 📝 `internal/net/gphttp/accesslog/access_logger_test.go` (+23 -1) 📝 `internal/net/gphttp/accesslog/formatter.go` (+30 -51) _...and 27 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

adam added the pull-request label 2025-12-29 09:23:44 +01:00

adam closed this issue

2025-12-29 09:23:44 +01:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/godoxy#165