Opened 7 years ago

Closed 3 years ago

#3167 closed bug (Fixed)

tracker_icons.py does not clean up /tmp entries from failed hosts

Reported by: rexar Owned by:
Priority: minor Milestone: 2.1.0
Component: GTK UI Version: 1.3.15
Keywords: icons tracker_icons Cc:

Description

If Deluge attempts to obtain a tracker icon for a tracker that is unreachable, or has a re-direct to self (perpetual HTTP 301/302) the errorback routines in tracker_icons.py do not have access to the temporary filehandles / paths created as part of the download_page method, and as such is unable to clean up these pre-allocated resources.

Over a period of time under Linux, this fills the /tmp directory with a large amount of junk files in the form of deluge_ticon.xxxxx

Attached is a workaround targetting the 1.3.15 release version of tracker_icons.py which allows the errorback routines to clean files allocated from failed attempts.

This was developed against my scenario which is the GTK client connecting to a remote server, it hasn't been tested for other use cases.

Attachments (1)

tracker_icons.patch (4.1 KB ) - added by rexar 7 years ago.
Patch file against 1.3.15 release version of tracker_icons.py

Download all attachments as: .zip

Change History (3)

by rexar, 7 years ago

Attachment: tracker_icons.patch added

Patch file against 1.3.15 release version of tracker_icons.py

comment:1 by Calum, 3 years ago

Milestone: needs verified2.1.0

Missed seeing this bug, should have been fixed, thanks for the workaround patch.

Duplicate reported in #3519

comment:2 by Calum, 3 years ago

Resolution: Fixed
Status: newclosed
Note: See TracTickets for help on using tickets.