[PR #304] [CLOSED] refactor(http_requests): Replace bufio.Scanner with bufio.Reader to i… #434

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

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-plugin-daemon/pull/304
Author: @homeant
Created: 5/23/2025
Status: Closed

Base: mainHead: fix-custom-tool-big-body


📝 Commits (2)

  • b2d7e9e refactor(http_requests): Replace bufio.Scanner with bufio.Reader to improve streaming data processing efficiency
  • b166310 Merge remote-tracking branch 'upstream/main' into fix-custom-tool-big-body

📊 Changes

1 file changed (+13 additions, -9 deletions)

View changed files

📝 internal/utils/http_requests/http_warpper.go (+13 -9)

📄 Description

refactor(http_requests): Replace bufio.Scanner with bufio.Reader to improve streaming data processing efficiency

Replaced bufio.Scanner with bufio.Reader in the RequestAndParseStream function to better handle errors in streaming data and enhance the stability of data reading. Additionally, optimized the logic for trimming and parsing data to reduce unnecessary operations.


🔄 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/304 **Author:** [@homeant](https://github.com/homeant) **Created:** 5/23/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix-custom-tool-big-body` --- ### 📝 Commits (2) - [`b2d7e9e`](https://github.com/langgenius/dify-plugin-daemon/commit/b2d7e9e55bf3eec2272f35415e026fd4d5886401) refactor(http_requests): Replace bufio.Scanner with bufio.Reader to improve streaming data processing efficiency - [`b166310`](https://github.com/langgenius/dify-plugin-daemon/commit/b1663102e9b7b923b8de523e57b33d14f73f3aed) Merge remote-tracking branch 'upstream/main' into fix-custom-tool-big-body ### 📊 Changes **1 file changed** (+13 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `internal/utils/http_requests/http_warpper.go` (+13 -9) </details> ### 📄 Description refactor(http_requests): Replace bufio.Scanner with bufio.Reader to improve streaming data processing efficiency Replaced bufio.Scanner with bufio.Reader in the RequestAndParseStream function to better handle errors in streaming data and enhance the stability of data reading. Additionally, optimized the logic for trimming and parsing data to reduce unnecessary operations. - fix langgenius/dify#307 - fix https://github.com/langgenius/dify-official-plugins/issues/1226 - fix https://github.com/langgenius/dify-official-plugins/issues/1002 - fix https://github.com/langgenius/dify-official-plugins/issues/1000 - fix https://github.com/langgenius/dify-official-plugins/issues/953 - fix https://github.com/langgenius/dify/issues/18501 - fix https://github.com/langgenius/dify/issues/19978 - fix https://github.com/langgenius/dify/issues/20270 --- <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:57 -05:00
yindo closed this issue 2026-02-16 01:15:57 -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#434