Opened 2 years ago

Last modified 6 months ago

#3505 new bug

Cannot add host on deluge-console (invalid port)

Reported by: tremby Owned by:
Priority: major Milestone: 2.1.1
Component: Console UI Version: 2.0.3
Keywords: fix Cc:


The "add host" option in deluge-console does not work.

I think the integer validation intended for the port field is checking the contents of the host field, or something similar.

If I type in a string in the host field and a number in the port field, I get "Error adding host -- localhost: Invalid port. Must be an integer".

Interestingly, if I fill the port number in to the host field, and the hostname in to the port field, I get "58846: Invalid port. Must be an integer".

Change History (5)

comment:1 Changed 2 years ago by Cas

  • Milestone changed from needs verified to 2.1.0

comment:2 Changed 2 years ago by KtlTheBest

Good time of the day. I am currently looking into the problem, but the quick analysis shows that there is no conversion of port value to integer.

There is a _do_add(result, **kwargs) method in ui/console/modes/ which then calls add_host(hostname, port, username, password) method from the same class, which in turn calls the add_host() from the Hostlist() class which is located in ui/ which then raises the Exception.

The question is, at what stage does the conversion from the str to int must happen?

Best regards, KtlTheBest?

comment:3 Changed 2 years ago by Cas

In the console connectionmanager I can see it is using add_text_input, could try replacing it with an int input field e.g. add_int_spin_input

comment:4 Changed 20 months ago by Cas

  • Milestone changed from 2.1.0 to 2.1.1

Ticket retargeted after milestone closed

comment:5 Changed 6 months ago by blob42

  • Keywords fix added

This fixed it for me:

   # cast port to int in ui/console/modes/
   self.hostlist.add_host(hostname, int(port), username, password)
Last edited 6 months ago by blob42 (previous) (diff)
Note: See TracTickets for help on using tickets.