[PR #176] [MERGED] Fix/lost query params in endpoint #348

Closed
opened 2026-02-16 01:15:39 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-plugin-daemon/pull/176
Author: @Yeuoly
Created: 4/7/2025
Status: Merged
Merged: 4/7/2025
Merged by: @Yeuoly

Base: mainHead: fix/lost-query-params-in-endpoint


📝 Commits (3)

  • eed91f2 feat: add X-Original-Host header handling in endpoint requests and improve request redirection URL construction
  • 9e676b7 feat: add test for redirecting traffic with query parameters and refactor URL construction logic
  • cddbad2 feat: add test for redirecting traffic without query parameters

📊 Changes

5 files changed (+67 additions, -1 deletions)

View changed files

📝 internal/cluster/redirect.go (+11 -1)
📝 internal/cluster/redirect_test.go (+39 -0)
📝 internal/server/endpoint.go (+6 -0)
📝 internal/service/endpoint.go (+7 -0)
📝 pkg/entities/endpoint_entities/endpoint.go (+4 -0)

📄 Description

Fixes
query parmas randomly lost
image


🔄 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/langgenius/dify-plugin-daemon/pull/176 **Author:** [@Yeuoly](https://github.com/Yeuoly) **Created:** 4/7/2025 **Status:** ✅ Merged **Merged:** 4/7/2025 **Merged by:** [@Yeuoly](https://github.com/Yeuoly) **Base:** `main` ← **Head:** `fix/lost-query-params-in-endpoint` --- ### 📝 Commits (3) - [`eed91f2`](https://github.com/langgenius/dify-plugin-daemon/commit/eed91f2359daaa0d32a706e1cbb4e0d846a93eee) feat: add X-Original-Host header handling in endpoint requests and improve request redirection URL construction - [`9e676b7`](https://github.com/langgenius/dify-plugin-daemon/commit/9e676b74684270367ff832c7adf2ca6819775cb4) feat: add test for redirecting traffic with query parameters and refactor URL construction logic - [`cddbad2`](https://github.com/langgenius/dify-plugin-daemon/commit/cddbad21bed1db0d737bbb5ea56d1026b1c93e40) feat: add test for redirecting traffic without query parameters ### 📊 Changes **5 files changed** (+67 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `internal/cluster/redirect.go` (+11 -1) 📝 `internal/cluster/redirect_test.go` (+39 -0) 📝 `internal/server/endpoint.go` (+6 -0) 📝 `internal/service/endpoint.go` (+7 -0) 📝 `pkg/entities/endpoint_entities/endpoint.go` (+4 -0) </details> ### 📄 Description Fixes query parmas randomly lost <img width="591" alt="image" src="https://github.com/user-attachments/assets/5cfcbca5-9a55-443c-91de-5c8db834f401" /> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 01:15:39 -05:00
yindo closed this issue 2026-02-16 01:15:39 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify-plugin-daemon#348