[pve-devel] applied: [PATCH manager v2] change icmp type selector to a combogrid
Thomas Lamprecht
t.lamprecht at proxmox.com
Wed Mar 16 15:23:51 CET 2022
On 29.05.20 14:22, Mira Limbeck wrote:
> The combogrid contains all valid icmp types that iptables accepts. In
> addition to the names, the Type[/Code] value is shown as well. But
> specifying Type[/Code] is not supported.
>
> As the simple solution with setStore() does not work to change the store
> for the combogrid, we simply have 2 different ICMPTypeSelectors, one for
> IPv4, the other for IPv6, which are switched depending on the specified
> protocol.
>
> As disabled fields don't submit their value, we can keep the values for
> the dport and both icmp-type fields when switching protocols.
>
> Signed-off-by: Mira Limbeck <m.limbeck at proxmox.com>
> ---
> v2:
> - rebased on master
> - removed unnecessary name switching and removing of values on proto
> switch
>
> www/manager6/grid/FirewallRules.js | 164 ++++++++++++++++++++++++++++-
> 1 file changed, 160 insertions(+), 4 deletions(-)
>
>
applied, thanks!
More information about the pve-devel
mailing list