HttpClientFactory doesn't clean up after itself

Open
#113,494 15 comments 4 reactions 1 assignee View on GitHub

Nobody has claimed this yet.

Assessment

This issue has not been assessed yet.

Description

area-Extensions-HttpClientFactory help wanted
Description

Recently I ran into an issue running tests using dotnet test that it would crash without any obvious reason.
Running my tests also showed memory would steadily increase over time whilst they were being run which indicated the presence of a memory leak.
Analysing the memory dump there were 1000s of timers holding on to objects that I would've expected to be disposed of all centered around ActiveHandlerTrackingEntry.

Reproduction Steps

Judging by the code, I'd say you could just get away with a for loop that creates a service collection, adds the http client factory, creates a service provider and then create a dozen or so http clients with it.

Expected behavior

When the service provider it's associated with is disposed of, all resources the http client factory is associated with should be disposed of and freed up, including any timers that it creates.

Actual behavior

Timers still hang around in the background, long after that instance of the http client factory is no longer in use. Which eventually consumes all available resources.

Regression?

No response

Known Workarounds

When running the tests ensuring the handler lifetime is set to infinite effectively bypasses the handler lifetime logic that creates the timers. Which stops the crashes that I was experiencing.

Configuration

Running .net 8.
MacOS and Linux, ARM64 and x64.
I don't think the configuration matters.

Other information

No response

Dominant language
C#
Stars
18.3k
Forks
5.6k
PR merge metrics
PR metrics pending

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from dotnet/runtime

All issues in dotnet/runtime

Similar issues

More C# issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.