[PR #118] [MERGED] fix: optimize memory usage, fix agent and code refactor #170

New Issue

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

adam commented

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

📋 Pull Request Information

Original PR: https://github.com/yusing/godoxy/pull/118
Author: @yusing
Created: 5/25/2025
Status: ✅ Merged
Merged: 5/25/2025
Merged by: @yusing

Base: main ← Head: dev

📝 Commits (10+)

f1e204f refactor: simplify io code and make utils module independent
df193a4 fix(docker): agent and socket-proxy docker events not flushing with modified reverse proxy handler
5a3e3f1 fix(imports): due to previous refactor
0f3b8e6 refactor: remove unused code
20a1649 tweak: disable bytes pool for socket-proxy and agent
63c4bdc refactor: remove the use of logging module in most code
1cf1865 refactor: streamline domain mismatch check in certState function
6a5cd12 tweak: use ecdsa p-256 for autocert
22d44a6 fix(tests): update health check tests for invalid host and add case for port in host
1f1ae38 feat(acme): custom acme directory

📊 Changes

98 files changed (+1548 additions, -554 deletions)

View changed files

📝 .golangci.yml (+5 -7)
📝 .trunk/trunk.yaml (+2 -2)
📝 README.md (+2 -0)
📝 README_CHT.md (+2 -0)
📝 agent/cmd/main.go (+15 -6)
📝 agent/go.mod (+3 -0)
📝 agent/pkg/agent/config.go (+4 -4)
📝 agent/pkg/handler/check_health_test.go (+17 -7)
📝 agent/pkg/handler/handler.go (+2 -25)
📝 agent/pkg/server/server.go (+2 -3)
📝 cmd/main.go (+6 -5)
📝 go.mod (+5 -2)
📝 internal/acl/config.go (+3 -3)
📝 internal/acl/matcher_test.go (+2 -2)
📝 internal/acl/tcp_listener.go (+3 -3)
📝 internal/api/v1/certapi/renew.go (+2 -2)
📝 internal/api/v1/dockerapi/logs.go (+3 -2)
📝 internal/auth/oauth_refresh.go (+6 -6)
📝 internal/auth/oidc.go (+11 -10)
📝 internal/autocert/config.go (+48 -11)

...and 78 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/118 **Author:** [@yusing](https://github.com/yusing) **Created:** 5/25/2025 **Status:** ✅ Merged **Merged:** 5/25/2025 **Merged by:** [@yusing](https://github.com/yusing) **Base:** `main` ← **Head:** `dev` --- ### 📝 Commits (10+) - [`f1e204f`](https://github.com/yusing/godoxy/commit/f1e204f7fd62e21e69c46151d4cdcd7acee01d08) refactor: simplify io code and make utils module independent - [`df193a4`](https://github.com/yusing/godoxy/commit/df193a42fc3f66438976f35c078e2b4388bab18d) fix(docker): agent and socket-proxy docker events not flushing with modified reverse proxy handler - [`5a3e3f1`](https://github.com/yusing/godoxy/commit/5a3e3f19c71076f6a42a86d56ea1c584b814393e) fix(imports): due to previous refactor - [`0f3b8e6`](https://github.com/yusing/godoxy/commit/0f3b8e68cebd339e07a5234c390e2cd9d27bbcdf) refactor: remove unused code - [`20a1649`](https://github.com/yusing/godoxy/commit/20a164927522ec3ab47833bb4e2ecc972cabfd5f) tweak: disable bytes pool for socket-proxy and agent - [`63c4bdc`](https://github.com/yusing/godoxy/commit/63c4bdc73d9059c2f9a3258c47c600da1e8f1195) refactor: remove the use of logging module in most code - [`1cf1865`](https://github.com/yusing/godoxy/commit/1cf18657b621337350ec965330934397a5436203) refactor: streamline domain mismatch check in certState function - [`6a5cd12`](https://github.com/yusing/godoxy/commit/6a5cd1266bf3208e26853e3b5fb9fd0d4e4d3542) tweak: use ecdsa p-256 for autocert - [`22d44a6`](https://github.com/yusing/godoxy/commit/22d44a6bb04c8aa9d8454ef9d80df282e6d7f193) fix(tests): update health check tests for invalid host and add case for port in host - [`1f1ae38`](https://github.com/yusing/godoxy/commit/1f1ae38e4d34806765af14e5a41011fd492cc877) feat(acme): custom acme directory ### 📊 Changes **98 files changed** (+1548 additions, -554 deletions) <details> <summary>View changed files</summary> 📝 `.golangci.yml` (+5 -7) 📝 `.trunk/trunk.yaml` (+2 -2) 📝 `README.md` (+2 -0) 📝 `README_CHT.md` (+2 -0) 📝 `agent/cmd/main.go` (+15 -6) 📝 `agent/go.mod` (+3 -0) 📝 `agent/pkg/agent/config.go` (+4 -4) 📝 `agent/pkg/handler/check_health_test.go` (+17 -7) 📝 `agent/pkg/handler/handler.go` (+2 -25) 📝 `agent/pkg/server/server.go` (+2 -3) 📝 `cmd/main.go` (+6 -5) 📝 `go.mod` (+5 -2) 📝 `internal/acl/config.go` (+3 -3) 📝 `internal/acl/matcher_test.go` (+2 -2) 📝 `internal/acl/tcp_listener.go` (+3 -3) 📝 `internal/api/v1/certapi/renew.go` (+2 -2) 📝 `internal/api/v1/dockerapi/logs.go` (+3 -2) 📝 `internal/auth/oauth_refresh.go` (+6 -6) 📝 `internal/auth/oidc.go` (+11 -10) 📝 `internal/autocert/config.go` (+48 -11) _...and 78 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:45 +01:00

adam closed this issue

2025-12-29 09:23:45 +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#170