Skip to content

Dispose image URL validation requests and responses - #70

Merged
LinFor merged 2 commits into
LinFor:masterfrom
Weegley:fix/remote-image-url-sanitizer
Jul 22, 2026
Merged

Dispose image URL validation requests and responses#70
LinFor merged 2 commits into
LinFor:masterfrom
Weegley:fix/remote-image-url-sanitizer

Conversation

@Weegley

@Weegley Weegley commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Dispose HttpRequestMessage and HttpResponseMessage instances created while validating remote image URLs.

This prevents connections from remaining in CLOSE_WAIT during metadata and image refreshes.

Changes

  • Dispose each image validation request.
  • Dispose each image validation response.
  • Add a regression test covering request and response disposal.

Testing

  • All 13 automated tests pass.
  • Tested with Jellyfin 10.11.11.
  • Performed a full metadata and image refresh for approximately 167 movies.
  • Tested newly added movies and repeated metadata replacement.
  • ESTABLISHED connections closed normally.
  • CLOSE_WAIT remained at or near zero.
  • No RemoteImageUrlSanitizer HTTP timeouts occurred.

@LinFor
LinFor merged commit aa0c6fe into LinFor:master Jul 22, 2026
1 check passed
@Weegley
Weegley deleted the fix/remote-image-url-sanitizer branch July 22, 2026 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants