starred/Cleanuparr
1
0
Fork 0
mirror of https://github.com/Cleanuparr/Cleanuparr.git synced 2026-04-13 21:25:37 -05:00
Code Issues 38 Packages Projects Releases 10 Wiki Activity

[PR #66] [MERGED] Fix interceptor memory leaks #274

New Issue
Closed
opened 2025-10-09 17:00:46 -05:00 by giteasync · 0 comments
giteasync commented 2025-10-09 17:00:46 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/Cleanuparr/Cleanuparr/pull/66
Author: @Flaminel
Created: 2/23/2025
Status: ✅ Merged
Merged: 2/23/2025
Merged by: @Flaminel

Base: main ← Head: fix_interceptor_memory_leaks

📝 Commits (5)

📊 Changes

29 files changed (+173 additions, -195 deletions)

View changed files

📝 code/Executable/DependencyInjection/MainDI.cs (+1 -31)
📝 code/Executable/DependencyInjection/NotificationsDI.cs (+1 -1)
📝 code/Executable/DependencyInjection/ServicesDI.cs (+1 -1)
📝 code/Infrastructure.Tests/Verticals/DownloadClient/DownloadServiceFixture.cs (+5 -2)
📝 code/Infrastructure.Tests/Verticals/DownloadClient/TestDownloadService.cs (+7 -3)
📝 code/Infrastructure/Infrastructure.csproj (+0 -1)
📝 code/Infrastructure/Interceptors/DryRunInterceptor.cs (+44 -16)
➕ code/Infrastructure/Interceptors/IDryRunInterceptor.cs (+10 -0)
➖ code/Infrastructure/Interceptors/IDryRunService.cs (+0 -5)
➖ code/Infrastructure/Interceptors/IInterceptedService.cs (+0 -6)
➖ code/Infrastructure/Interceptors/InterceptedService.cs (+0 -21)
📝 code/Infrastructure/Verticals/Arr/ArrClient.cs (+7 -10)
📝 code/Infrastructure/Verticals/Arr/LidarrClient.cs (+6 -8)
📝 code/Infrastructure/Verticals/Arr/RadarrClient.cs (+6 -8)
📝 code/Infrastructure/Verticals/Arr/SonarrClient.cs (+7 -9)
📝 code/Infrastructure/Verticals/ContentBlocker/ContentBlocker.cs (+1 -1)
📝 code/Infrastructure/Verticals/DownloadCleaner/DownloadCleaner.cs (+1 -1)
📝 code/Infrastructure/Verticals/DownloadClient/Deluge/DelugeService.cs (+10 -11)
📝 code/Infrastructure/Verticals/DownloadClient/DownloadService.cs (+7 -10)
📝 code/Infrastructure/Verticals/DownloadClient/DummyDownloadService.cs (+2 -6)

...and 9 more files

📄 Description

No description provided

🔄 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/Cleanuparr/Cleanuparr/pull/66 **Author:** [@Flaminel](https://github.com/Flaminel) **Created:** 2/23/2025 **Status:** ✅ Merged **Merged:** 2/23/2025 **Merged by:** [@Flaminel](https://github.com/Flaminel) **Base:** `main` ← **Head:** `fix_interceptor_memory_leaks` --- ### 📝 Commits (5) - [`c9fbd85`](https://github.com/Cleanuparr/Cleanuparr/commit/c9fbd859d54f46c69096ca42c03517510a812cfa) removed proxy - [`9df9b46`](https://github.com/Cleanuparr/Cleanuparr/commit/9df9b46d69d3ecef8883929ae17b5ec0cc6441a4) removed Castle package due to memory leaks - [`9df5381`](https://github.com/Cleanuparr/Cleanuparr/commit/9df53812fe0068b4ea42f797a74e633ba472097d) fixed dry run interceptor - [`4d8f915`](https://github.com/Cleanuparr/Cleanuparr/commit/4d8f9151b20b935bb49a05d72c8185e28dc1245c) removed comment - [`b00cc91`](https://github.com/Cleanuparr/Cleanuparr/commit/b00cc914cb894c0dd65f42a2d717a6ef50091275) fixed interceptor return type ### 📊 Changes **29 files changed** (+173 additions, -195 deletions) <details> <summary>View changed files</summary> 📝 `code/Executable/DependencyInjection/MainDI.cs` (+1 -31) 📝 `code/Executable/DependencyInjection/NotificationsDI.cs` (+1 -1) 📝 `code/Executable/DependencyInjection/ServicesDI.cs` (+1 -1) 📝 `code/Infrastructure.Tests/Verticals/DownloadClient/DownloadServiceFixture.cs` (+5 -2) 📝 `code/Infrastructure.Tests/Verticals/DownloadClient/TestDownloadService.cs` (+7 -3) 📝 `code/Infrastructure/Infrastructure.csproj` (+0 -1) 📝 `code/Infrastructure/Interceptors/DryRunInterceptor.cs` (+44 -16) ➕ `code/Infrastructure/Interceptors/IDryRunInterceptor.cs` (+10 -0) ➖ `code/Infrastructure/Interceptors/IDryRunService.cs` (+0 -5) ➖ `code/Infrastructure/Interceptors/IInterceptedService.cs` (+0 -6) ➖ `code/Infrastructure/Interceptors/InterceptedService.cs` (+0 -21) 📝 `code/Infrastructure/Verticals/Arr/ArrClient.cs` (+7 -10) 📝 `code/Infrastructure/Verticals/Arr/LidarrClient.cs` (+6 -8) 📝 `code/Infrastructure/Verticals/Arr/RadarrClient.cs` (+6 -8) 📝 `code/Infrastructure/Verticals/Arr/SonarrClient.cs` (+7 -9) 📝 `code/Infrastructure/Verticals/ContentBlocker/ContentBlocker.cs` (+1 -1) 📝 `code/Infrastructure/Verticals/DownloadCleaner/DownloadCleaner.cs` (+1 -1) 📝 `code/Infrastructure/Verticals/DownloadClient/Deluge/DelugeService.cs` (+10 -11) 📝 `code/Infrastructure/Verticals/DownloadClient/DownloadService.cs` (+7 -10) 📝 `code/Infrastructure/Verticals/DownloadClient/DummyDownloadService.cs` (+2 -6) _...and 9 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
giteasync added the pull-request label 2025-10-09 17:00:46 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
enhancement
enhancement
planned
pull-request
Mirrored from GitHub Pull Request
 question
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
giteasync
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/Cleanuparr#274
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?