[PR #3547] [MERGED] fix a goroutine leak issue caused by Login plugin timeout #4778

Closed
opened 2026-05-05 14:48:32 -06:00 by gitea-mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/fatedier/frp/pull/3547
Author: @fatedier
Created: 7/25/2023
Status: Merged
Merged: 7/25/2023
Merged by: @fatedier

Base: devHead: new


📝 Commits (1)

  • 2e58355 fix a goroutine leak issue caused by Login plugin timeout

📊 Changes

3 files changed (+30 additions, -12 deletions)

View changed files

📝 Release.md (+4 -0)
📝 server/control.go (+25 -11)
📝 server/service.go (+1 -1)

📄 Description

Summary

🤖 Generated by Copilot at f40bbd9

This pull request improves the server-side logic and performance of the frp project. It fixes a bug that caused a goroutine leak when using the Login plugin, removes unused or redundant code, and enhances the Control type and its methods for handling client connections. It also updates the Release.md file to document the bug fix.

WHY

Fix #3543


🔄 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/fatedier/frp/pull/3547 **Author:** [@fatedier](https://github.com/fatedier) **Created:** 7/25/2023 **Status:** ✅ Merged **Merged:** 7/25/2023 **Merged by:** [@fatedier](https://github.com/fatedier) **Base:** `dev` ← **Head:** `new` --- ### 📝 Commits (1) - [`2e58355`](https://github.com/fatedier/frp/commit/2e583555e8f145a09b09497dcd37fc3315483105) fix a goroutine leak issue caused by Login plugin timeout ### 📊 Changes **3 files changed** (+30 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `Release.md` (+4 -0) 📝 `server/control.go` (+25 -11) 📝 `server/service.go` (+1 -1) </details> ### 📄 Description ### Summary <!-- copilot:summary --> ### <samp>🤖 Generated by Copilot at f40bbd9</samp> This pull request improves the server-side logic and performance of the frp project. It fixes a bug that caused a goroutine leak when using the Login plugin, removes unused or redundant code, and enhances the `Control` type and its methods for handling client connections. It also updates the `Release.md` file to document the bug fix. ### WHY Fix #3543 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
gitea-mirror 2026-05-05 14:48:32 -06:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/frp#4778
No description provided.