Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#2948 closed bug (Fixed)

Non-Ascii Character Breaks Console Torrent Remove

Reported by: coreyme Owned by:
Priority: minor Milestone: 1.3.14
Component: Console UI Version: 1.3.13
Keywords: ascii Cc:

Description

If a torrent with a non-ascii character like an En Dash in its name is running, the console command "deluge-console.exe rm --remove_data <id>" will fail for ANY torrent and returns the message:

'ascii' codec can't decode byte 0xe2 in position 9: ordinal not in range(128)
[ERROR   ] 13:48:42 main:347 'ascii' codec can't decode byte 0xe2 in position 9: ordinal not in range(128)
Traceback (most recent call last):
  File "deluge\ui\console\main.py", line 344, in do_command
  File "deluge\ui\console\commands\rm.py", line 61, in handle
  File "deluge\ui\console\main.py", line 462, in match_torrent
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 9: ordinal not in range(128)
Traceback (most recent call last):
  File "deluge\ui\console\main.py", line 344, in do_command
  File "deluge\ui\console\commands\rm.py", line 61, in handle
  File "deluge\ui\console\main.py", line 462, in match_torrent
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 9: ordinal not in range(128)

Removing the torrent with the non-ascii character resolves the problem.

Change History (5)

comment:1 by Calum, 8 years ago

Milestone: needs verified1.3.14

comment:2 by Calum, 8 years ago

This is odd because there was a fix applied: [5d0dace1317ac] for #2045

So I shall I have to try to replicate myself and see if our own decode function works.

Last edited 8 years ago by Calum (previous) (diff)

comment:3 by coreyme, 8 years ago

Thanks! If it matters, I'm running Deluge 1.3.13 on Windows 10 x64 with my language set to English (Canada).

comment:4 by Calum, 8 years ago

Resolution: Fixed
Status: newclosed

Ah it was only a half-fix for #2045, that's the other bit done now :)

Fixed 1.3-stable: [30eaf775c2d5]

comment:5 by coreyme, 8 years ago

Thanks, Cas!

Note: See TracTickets for help on using tickets.