Modify

Opened 6 years ago

Last modified 4 years ago

#11417 new enhancement

luci-qos: usability improvements by better "Ports" row description

Reported by: Jens <nachtigall@…> Owned by:
Priority: low Milestone: Barrier Breaker 14.07
Component: luci Version: Trunk
Keywords: qos, luci, ports, iptables Cc:

Description

two simple things. This should either be handled by the (luci-)qos package or at least the user be told about it:

I had two big (simple) problems:

  • a line like "9962, 9965, 9972" in the "Ports" row results in an error like this on command line:
# /etc/init.d/qos start 
iptables v1.4.6: invalid port/service `' specified
Try `iptables -h' or 'iptables --help' for more information.
iptables v1.4.6: invalid port/service `' specified
Try `iptables -h' or 'iptables --help' for more information.

Reason are the spaces between the items.

  • also if you have more than x ports in the Ports row, then iptables will complain because it can only handle a certain number of Ports per command line call (I think 12 or something similar). Either luci should split to long lines up to at least the maximum allowed number of iptables. Or the user should be told about the syntax and max length of the Ports row.

The worst is that the failure of /etc/init.d/qos restart becomes obvious only from command line, but not if one uses the luci webinterface. That is, there should be some kind of feedback (e.g. the command line output), otherwise one thinks that luci-qos was successfully restarted which it is actually not.

Attachments (0)

Change History (1)

comment:1 Changed 4 years ago by jow

  • Milestone changed from Attitude Adjustment 12.09 to Barrier Breaker 14.07

Milestone Attitude Adjustment 12.09 deleted

Add Comment

Modify Ticket

Action
as new .
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.