[PR #160] [MERGED] post request rules and overhaul #173

New Issue

adam · 2025-12-29T15:18:51+01:00

adam commented

2025-12-29 15:18:51 +01:00

📋 Pull Request Information

Original PR: https://github.com/yusing/godoxy/pull/160
Author: @yusing
Created: 10/13/2025
Status: ✅ Merged
Merged: 10/14/2025
Merged by: @yusing

Base: main ← Head: feat/post-req-rules

📝 Commits (10+)

54cfa21 tmp: post-req rules
3f45f3c tmp: post-req-rules-fix
91dc40a Merge branch 'main' into feat/post-req-rules
3d60fab feat: add post-request rules system with response manipulation
4dc6bd3 fix(rules): parser dollar sign and env handling
75c074d fix(rules): correct behavior of require_auth
16b986d fix(rules): proxying relative url should use upstream host instead of request host
f93292f feat(rules): support negated conditions with !
df62c26 fix: remove debug logging
0852f91 fix(rules): validation, error handling and add missing tests

📊 Changes

41 files changed (+4419 additions, -522 deletions)

View changed files

📝 goutils (+1 -1)
📝 internal/auth/auth.go (+10 -0)
📝 internal/logging/accesslog/config.go (+1 -1)
📝 internal/logging/accesslog/file_logger.go (+4 -1)
📝 internal/logging/accesslog/file_logger_test.go (+1 -1)
📝 internal/logging/logging.go (+20 -0)
📝 internal/net/gphttp/middleware/bypass.go (+5 -7)
📝 internal/notif/body.go (+14 -5)
📝 internal/route/reverse_proxy.go (+1 -1)
📝 internal/route/route.go (+45 -2)
📝 internal/route/routes/context.go (+7 -0)
📝 internal/route/rules/cache.go (+15 -15)
📝 internal/route/rules/check_on.go (+8 -8)
📝 internal/route/rules/command.go (+46 -18)
📝 internal/route/rules/do.go (+287 -29)
➕ internal/route/rules/do_log_test.go (+400 -0)
➕ internal/route/rules/do_set.go (+328 -0)
➕ internal/route/rules/do_set_test.go (+643 -0)
📝 internal/route/rules/do_test.go (+6 -1)
➕ internal/route/rules/error_format_test.go (+23 -0)

...and 21 more files

📄 Description

tmp: post-req rules
tmp: post-req-rules-fix
feat: add post-request rules system with response manipulation
fix(rules): parser dollar sign and env handling
fix(rules): correct behavior of require_auth
fix(rules): proxying relative url should use upstream host instead of request host
feat(rules): support negated conditions with !

_{🔄 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/160 **Author:** [@yusing](https://github.com/yusing) **Created:** 10/13/2025 **Status:** ✅ Merged **Merged:** 10/14/2025 **Merged by:** [@yusing](https://github.com/yusing) **Base:** `main` ← **Head:** `feat/post-req-rules` --- ### 📝 Commits (10+) - [`54cfa21`](https://github.com/yusing/godoxy/commit/54cfa21f2bde04bb364833c67306527053e44b61) tmp: post-req rules - [`3f45f3c`](https://github.com/yusing/godoxy/commit/3f45f3c4e20dbc85c8b532d41f3e25129cf3c055) tmp: post-req-rules-fix - [`91dc40a`](https://github.com/yusing/godoxy/commit/91dc40adb0176be3b708967d0fa2198d3b75df32) Merge branch 'main' into feat/post-req-rules - [`3d60fab`](https://github.com/yusing/godoxy/commit/3d60fabc5ef5a9dc0cd8eb330972adb26837ccd1) feat: add post-request rules system with response manipulation - [`4dc6bd3`](https://github.com/yusing/godoxy/commit/4dc6bd3dd1369aeab6a65a6766a88df43af8e7b5) fix(rules): parser dollar sign and env handling - [`75c074d`](https://github.com/yusing/godoxy/commit/75c074dd60425f1902d5df9a974ddea9f9de063e) fix(rules): correct behavior of require_auth - [`16b986d`](https://github.com/yusing/godoxy/commit/16b986d86bd58aed3778e1fb88e4d16a8b3b8e60) fix(rules): proxying relative url should use upstream host instead of request host - [`f93292f`](https://github.com/yusing/godoxy/commit/f93292f2a1f7d5f986bd6087b07b726ddf52ec0a) feat(rules): support negated conditions with ! - [`df62c26`](https://github.com/yusing/godoxy/commit/df62c267aeb3f49c2197dc680e065159377e384b) fix: remove debug logging - [`0852f91`](https://github.com/yusing/godoxy/commit/0852f918df1a1adb48c18504d5a24f9c92efd773) fix(rules): validation, error handling and add missing tests ### 📊 Changes **41 files changed** (+4419 additions, -522 deletions) <details> <summary>View changed files</summary> 📝 `goutils` (+1 -1) 📝 `internal/auth/auth.go` (+10 -0) 📝 `internal/logging/accesslog/config.go` (+1 -1) 📝 `internal/logging/accesslog/file_logger.go` (+4 -1) 📝 `internal/logging/accesslog/file_logger_test.go` (+1 -1) 📝 `internal/logging/logging.go` (+20 -0) 📝 `internal/net/gphttp/middleware/bypass.go` (+5 -7) 📝 `internal/notif/body.go` (+14 -5) 📝 `internal/route/reverse_proxy.go` (+1 -1) 📝 `internal/route/route.go` (+45 -2) 📝 `internal/route/routes/context.go` (+7 -0) 📝 `internal/route/rules/cache.go` (+15 -15) 📝 `internal/route/rules/check_on.go` (+8 -8) 📝 `internal/route/rules/command.go` (+46 -18) 📝 `internal/route/rules/do.go` (+287 -29) ➕ `internal/route/rules/do_log_test.go` (+400 -0) ➕ `internal/route/rules/do_set.go` (+328 -0) ➕ `internal/route/rules/do_set_test.go` (+643 -0) 📝 `internal/route/rules/do_test.go` (+6 -1) ➕ `internal/route/rules/error_format_test.go` (+23 -0) _...and 21 more files_ </details> ### 📄 Description - **tmp: post-req rules** - **tmp: post-req-rules-fix** - **feat: add post-request rules system with response manipulation** - **fix(rules): parser dollar sign and env handling** - **fix(rules): correct behavior of require_auth** - **fix(rules): proxying relative url should use upstream host instead of request host** - **feat(rules): support negated conditions with !** --- <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 15:18:51 +01:00

adam closed this issue

2025-12-29 15:18:52 +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#173