Memory leak in DefaultPartitionedRateLimiter - backport to NET10
Open
Beginner friendly
area-System.Threading
untriaged
- Dominant language
- C#
- Stars
- 18.3k
- Forks
- 5.6k
- PR merge metrics
- PR metrics pending
Description
As reported in https://github.com/dotnet/runtime/issues/124557, `DefaultPartitionedRateLimiter` doesn't dispose of `NoopLimiter` instances, causing a memory leak.
This issue was fixed in https://github.com/dotnet/runtime/pull/127582, but only merged to `main`; never backported to net10, which is supposed to be supported until Nov 2028.
Can the fix be applied to the NET10 branch as well please? The rate limiter is currently unusable otherwise.
Contributor guide
Research direction
Start by reviewing issue 124557 and pull request 127582, then compare the merged fix with the NET10 branch. Apply the existing fix to NET10 and run the relevant rate limiter tests to confirm that the memory leak is addressed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- performance
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 68/100