[PR #60] [MERGED] Feat/fileserver #154

New Issue

adam · 2025-12-29T14:25:43+01:00

adam commented

2025-12-29 14:25:43 +01:00

📋 Pull Request Information

Original PR: https://github.com/yusing/godoxy/pull/60
Author: @yusing
Created: 2/6/2025
Status: ✅ Merged
Merged: 2/6/2025
Merged by: @yusing

Base: v0.9 ← Head: feat/fileserver

📝 Commits (10+)

721ddcb cleanup code for URL type
58d5e81 fix makefile for trace mode
c61d893 refactor, merge Entry, RawEntry and Route into one. Implement fileserver. Still buggy
1baf10f fix incorrect route loading behaviors
ec148f0 treat fileserver as http route
503671b refactor: rename HTTPRoute to ReverseProxyRoute to avoid confusion
1e413ae refactor: move metrics logger to middleware package
c90e0db reverse proxy refactor and bug fixes
199a4bb fix error formatting
1d4d2a0 clearer error message when config reload failed

📊 Changes

79 files changed (+1477 additions, -1269 deletions)

View changed files

📝 .golangci.yml (+2 -4)
📝 .trunk/trunk.yaml (+5 -5)
📝 Makefile (+1 -0)
📝 cmd/main.go (+5 -4)
📝 internal/api/v1/auth/oidc_test.go (+1 -2)
📝 internal/api/v1/auth/userpass_test.go (+3 -4)
📝 internal/api/v1/favicon/cache.go (+4 -1)
📝 internal/api/v1/favicon/favicon.go (+2 -2)
📝 internal/config/query.go (+4 -5)
📝 internal/docker/container.go (+1 -1)
📝 internal/docker/container_helper.go (+5 -11)
📝 internal/docker/idlewatcher/waker.go (+12 -12)
📝 internal/docker/idlewatcher/watcher.go (+2 -2)
📝 internal/entrypoint/entrypoint_test.go (+1 -1)
📝 internal/error/utils.go (+0 -7)
📝 internal/homepage/override_config.go (+9 -7)
📝 internal/list-icons.go (+3 -0)
📝 internal/net/http/accesslog/access_logger_test.go (+1 -2)
📝 internal/net/http/loadbalancer/types/server.go (+5 -5)
➕ internal/net/http/middleware/metrics_logger/metrics_logger.go (+44 -0)

...and 59 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/60 **Author:** [@yusing](https://github.com/yusing) **Created:** 2/6/2025 **Status:** ✅ Merged **Merged:** 2/6/2025 **Merged by:** [@yusing](https://github.com/yusing) **Base:** `v0.9` ← **Head:** `feat/fileserver` --- ### 📝 Commits (10+) - [`721ddcb`](https://github.com/yusing/godoxy/commit/721ddcb590339b2826dafe4150ac00f808376382) cleanup code for URL type - [`58d5e81`](https://github.com/yusing/godoxy/commit/58d5e8144956358e04d323b10b47e8623912f80c) fix makefile for trace mode - [`c61d893`](https://github.com/yusing/godoxy/commit/c61d893403a2201241880f55f260d85c64956c02) refactor, merge Entry, RawEntry and Route into one. Implement fileserver. Still buggy - [`1baf10f`](https://github.com/yusing/godoxy/commit/1baf10fa88ae0f16761a1af62cdc64d90c3905df) fix incorrect route loading behaviors - [`ec148f0`](https://github.com/yusing/godoxy/commit/ec148f036d95c3681a70cf06526578ee7715302a) treat fileserver as http route - [`503671b`](https://github.com/yusing/godoxy/commit/503671bd1b91732d3ae4beda5d82fc474eccad6c) refactor: rename HTTPRoute to ReverseProxyRoute to avoid confusion - [`1e413ae`](https://github.com/yusing/godoxy/commit/1e413ae215cf9e081085bca35fad5049b8c310d1) refactor: move metrics logger to middleware package - [`c90e0db`](https://github.com/yusing/godoxy/commit/c90e0db30d1a6916220d02a0c502126118f4e8db) reverse proxy refactor and bug fixes - [`199a4bb`](https://github.com/yusing/godoxy/commit/199a4bb876b3b54fc5e86b77b0ad98ece4c729fb) fix error formatting - [`1d4d2a0`](https://github.com/yusing/godoxy/commit/1d4d2a02ed513879e6e8ab29d8e86d7af1dc49d0) clearer error message when config reload failed ### 📊 Changes **79 files changed** (+1477 additions, -1269 deletions) <details> <summary>View changed files</summary> 📝 `.golangci.yml` (+2 -4) 📝 `.trunk/trunk.yaml` (+5 -5) 📝 `Makefile` (+1 -0) 📝 `cmd/main.go` (+5 -4) 📝 `internal/api/v1/auth/oidc_test.go` (+1 -2) 📝 `internal/api/v1/auth/userpass_test.go` (+3 -4) 📝 `internal/api/v1/favicon/cache.go` (+4 -1) 📝 `internal/api/v1/favicon/favicon.go` (+2 -2) 📝 `internal/config/query.go` (+4 -5) 📝 `internal/docker/container.go` (+1 -1) 📝 `internal/docker/container_helper.go` (+5 -11) 📝 `internal/docker/idlewatcher/waker.go` (+12 -12) 📝 `internal/docker/idlewatcher/watcher.go` (+2 -2) 📝 `internal/entrypoint/entrypoint_test.go` (+1 -1) 📝 `internal/error/utils.go` (+0 -7) 📝 `internal/homepage/override_config.go` (+9 -7) 📝 `internal/list-icons.go` (+3 -0) 📝 `internal/net/http/accesslog/access_logger_test.go` (+1 -2) 📝 `internal/net/http/loadbalancer/types/server.go` (+5 -5) ➕ `internal/net/http/middleware/metrics_logger/metrics_logger.go` (+44 -0) _...and 59 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 14:25:43 +01:00

adam closed this issue

2025-12-29 14:25:43 +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-yusing#154