Modify

Opened 5 years ago

Last modified 5 years ago

#13608 reopened enhancement

Add TCP+SCTP config option to UCI and LuCI.

Reported by: james.r.haigh@… Owned by:
Priority: normal Milestone:
Component: luci Version:
Keywords: sctp uci luci Cc:

Description

I want to prevent UDP, at least on the downlink, but replace it with SCTP due to its congestion control. That means I want to replace the TCP+UDP (tcpudp) option with ‘TCP+SCTP’ in my configuration.

According to the firewall documentation, the value of proto can only be “one of tcp, udp, tcpudp, udplite, icmp, esp, ah, sctp, or all or it can be a numeric value, representing one of these protocols or a different one”. Surely it makes sense to use a comma-separated list and deprecate the value tcpudp which will be equivalent to tcp,udp. The value I want will be tcp,sctp.

LuCI should also reflect this, either by adding TCP+UDP+SCTP, TCP+SCTP, and SCTP to the drop-down list, or by using check-boxes or some other interface feature that involves selection.

Attachments (0)

Change History (5)

comment:1 Changed 5 years ago by jow

  • Resolution set to worksforme
  • Status changed from new to closed

Simply choose "custom" and enter "tcp sctp".

comment:2 Changed 5 years ago by James Haigh <james.r.haigh@…>

  • Resolution worksforme deleted
  • Status changed from closed to reopened

jow: Using a space-separated list seems to work for me too. Thanks for this. Could you tell me which version introduced this, so that I can update the wiki page with this info.

That clears up the UCI side, but LuCI still needs some changes. Please read the last sentence of the description. Using custom is inadequate because it doesn't allow ports to be specified. Even if a port is already specified in the config file, LuCI will remove this upon any change. Please either add TCP+UDP+SCTP, TCP+SCTP, and SCTP to the drop-down list, or use something like checkboxes.

BTW, do you know if there is a way of allowing SCTP over UDP but blocking all other UDP packets?

comment:3 Changed 5 years ago by James Haigh <james.r.haigh@…>

comment:4 Changed 5 years ago by James Haigh <james.r.haigh@…>

The checkboxes idea would be neater than having all 6 combinations of TCP, SCTP, and UDP, as well as custom and blank / current custom (7 or 8 total). Currently there are 4 or 5 list items. The checkboxes idea is ideal and would look like this:

☑ TCP
☐ UDP
☑ SCTP
custom: tcp sctp

Note that custom would automatically reflect the value of the checkboxes and vice versa.

comment:5 Changed 5 years ago by James Haigh <james.r.haigh@…>

Oh, and for rules, it would look like this:

☑ Any
☐ TCP
☐ UDP
☐ SCTP
☐ ICMP
custom: all

Add Comment

Modify Ticket

Action
as reopened .
Author


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

 
Note: See TracTickets for help on using tickets.