[PR #518] [MERGED] fix: fix ReadCloser is not closed #557

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

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-plugin-daemon/pull/518
Author: @fatelei
Created: 12/3/2025
Status: Merged
Merged: 12/3/2025
Merged by: @fatelei

Base: mainHead: unclose


📝 Commits (1)

  • 62b48ed fix: fix ReadCloser is not closed

📊 Changes

2 files changed (+8 additions, -13 deletions)

View changed files

📝 internal/cluster/redirect_test.go (+1 -0)
📝 internal/server/middleware.go (+7 -13)

📄 Description

Description

ReadCloser is close #519

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Performance improvement
  • Other

Essential Checklist

Testing

  • I have tested the changes locally and confirmed they work as expected
  • I have added unit tests where necessary and they pass successfully

Bug Fix (if applicable)

  • I have used GitHub syntax to close the related issue (e.g., Fixes #123 or Closes #123)

Additional Information

ReadCloser is close


🔄 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/518 **Author:** [@fatelei](https://github.com/fatelei) **Created:** 12/3/2025 **Status:** ✅ Merged **Merged:** 12/3/2025 **Merged by:** [@fatelei](https://github.com/fatelei) **Base:** `main` ← **Head:** `unclose` --- ### 📝 Commits (1) - [`62b48ed`](https://github.com/langgenius/dify-plugin-daemon/commit/62b48ed1aa6aa792c42464ed9b20d8a6557de94d) fix: fix ReadCloser is not closed ### 📊 Changes **2 files changed** (+8 additions, -13 deletions) <details> <summary>View changed files</summary> 📝 `internal/cluster/redirect_test.go` (+1 -0) 📝 `internal/server/middleware.go` (+7 -13) </details> ### 📄 Description ## Description ReadCloser is close #519 ## Type of Change - [x] Bug fix - [ ] New feature - [ ] Refactor - [ ] Performance improvement - [ ] Other ## Essential Checklist ### Testing - [x] I have tested the changes locally and confirmed they work as expected - [ ] I have added unit tests where necessary and they pass successfully ### Bug Fix (if applicable) - [x] I have used GitHub syntax to close the related issue (e.g., `Fixes #123` or `Closes #123`) ## Additional Information ReadCloser is close --- <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:16:19 -05:00
yindo closed this issue 2026-02-16 01:16:19 -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#557