mirror/UNIT3D-Community-Edition
1
0
Fork 0
mirror of https://github.com/HDInnovations/UNIT3D-Community-Edition.git synced 2026-04-21 01:00:08 -05:00
Code Issues Packages Projects Releases Wiki Activity

fix: edge case resulting in double announce traffic stats

Browse Source 
When a user

1) completes a torrent (downloaded = x),
2) immediately stops it (downloaded = x),
3) resumes it (downloaded = 0) within the next 30 seconds,

then, during (1), the downloaded delta will be x, but (2) is considered overlapping (1), so it is delayed 30 seconds, by which time (3) will get processed first, having a delta of 0. After the 30 seconds are up, (2) will be processed, but since the previous processed announce (3) had a downloaded of 0, its delta will be x, causing the download to be doubled.

The same happens with upload.
This commit is contained in:
Roardom
2024-02-23 05:41:08 +00:00
parent 9dd1e5d0ee
commit ea41dbedcc
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/Jobs/ProcessAnnounce.php
+1 -1
View File
@@ -55,7 +55,7 @@ class ProcessAnnounce implements ShouldQueue
*/
public function middleware(): array
{
return [(new WithoutOverlapping($this->user->id.':'.$this->torrent->id))->releaseAfter(30)];
return [new WithoutOverlapping($this->user->id.':'.$this->torrent->id)];
}
/**