joshpatra/allstarr
1
0
Fork 0
mirror of https://github.com/SoPat712/allstarr.git synced 2026-02-09 23:55:10 -05:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

fix: prevent duplicate downloads by registering before releasing lock

Browse Source 
- Move RegisterDownloadedSongAsync call before lock release
- Prevents race condition where multiple threads download same song
- Second thread now finds registered mapping before starting download
- Fixes duplicate files like 'Song (1).flac', 'Song (2).flac'
- Library scan and album download still happen after lock release
This commit is contained in:
Josh Patra
2026-01-30 13:37:42 -05:00
parent e8d3fc4d17
commit df8dbfc5e1
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
allstarr/Services/Common/BaseDownloadService.cs
View File

@@ -298,6 +298,12 @@ public abstract class BaseDownloadService : IDownloadService
song.LocalPath = localPath; song.LocalPath = localPath;
// Register BEFORE releasing lock to prevent race conditions
if (!isCache)
{
await LocalLibraryService.RegisterDownloadedSongAsync(song, localPath);
}
// Check if this track belongs to a playlist and update M3U // Check if this track belongs to a playlist and update M3U
if (PlaylistSyncService != null) if (PlaylistSyncService != null)
{ {
@@ -316,11 +322,9 @@ public abstract class BaseDownloadService : IDownloadService
} }
} }
// Only register and scan if NOT in cache mode // Trigger library scan and album download AFTER releasing lock
if (!isCache) if (!isCache)
{ {
await LocalLibraryService.RegisterDownloadedSongAsync(song, localPath);
// Trigger a Subsonic library rescan (with debounce) // Trigger a Subsonic library rescan (with debounce)
_ = Task.Run(async () => _ = Task.Run(async () =>
{ {