On 8/19/25 2:47 PM, Pablo Neira Ayuso wrote:
Hi,
I made a changes to this series and push it out to the 'tunnel' branch
under the nftables tree.
I added IPv6 support to this 1/7 patch.
However:
[PATCH 6/7] tunnel: add tunnel object and statement json support
still needs to be adjusted to also support IPv6.
I can see JSON representation uses "src" and "dst" as keys.
It is better to have keys that uniquely identify the datatype, I would
suggest:
"ip saddr"
"ip6 saddr"
(similar to ct expression)
or
"src-ipv4"
"src-ipv6"
Hi Pablo,
I would prefer the latter options "src-ipv4" and "src-ipv6" mainly
because adding spaces to keys in a JSON should be avoided.
I am sending a v3 with the requested changes, thank you!
else, anything you consider better for this.
I think this is the only remaining issue in this series IMO.
If you follow up, patch based your work on the two branches that I
have pushed out for libnftnl and nftables.
Thanks.
