[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