Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NFT support #45

Open
willyhun opened this issue Oct 27, 2015 · 13 comments
Open

NFT support #45

willyhun opened this issue Oct 27, 2015 · 13 comments
Labels

Comments

@willyhun
Copy link

Do you plan support nftables infrastructure? Thanks!

@aabc
Copy link
Owner

aabc commented Oct 27, 2015

As far as I know, nftables is not extensible.

@aabc aabc added the invalid label Oct 28, 2015
@aabc
Copy link
Owner

aabc commented Oct 29, 2015

Well, if you (or someone else) find a way (or example how) to extend nftables, please send!

@q2dg
Copy link

q2dg commented Feb 24, 2019

@aabc
Copy link
Owner

aabc commented Feb 28, 2019

@q2dg Thanks. That's interesting. But he still patch libnftnl and nftables userspace to support new statement.

@junjunk
Copy link

junjunk commented Mar 26, 2019

Made fork and added support for nftables
https://github.com/junjunk/ipt-netflow/tree/nftables
This is the first testing commit - no checks in configure
Userspace patches will be a bit later

@aabc
Copy link
Owner

aabc commented Mar 27, 2019

@junjunk Thanks for working on it. It seems break compatibility with iptables too?

@junjunk
Copy link

junjunk commented Mar 27, 2019

Well, I tried to leave compatibility with iptables untouched.
Init function of module tries to register 2 different subsys - first with xt_register_targets(ipt_netflow_reg, ARRAY_SIZE(ipt_netflow_reg)) as it was, and second one with nft_register_expr(&nft_netflow_type)
Also, I've made single main function netflow_target with 2 wrappers ipt_netflow_target for iptables and nft_netflow_eval for nftables

@junjunk
Copy link

junjunk commented Mar 27, 2019

Also, it seems to me that better way to register iptables/nftables targets via module params

@aabc
Copy link
Owner

aabc commented Mar 28, 2019

Ah, that sounds good. (I was misinterpreted the patch, probably.)

@stalker37
Copy link

what plans to add support of NFT?

@aabc
Copy link
Owner

aabc commented May 17, 2019

@stalker37 There is no way to make netflow target as a separate installable module for nft. nftables source code patching is needed. But they will (probably) not integrate it into nftables because it is not in the kernel upstream. So, nftables is less flexible in the sense of extensibility than iptables.

@stalker37
Copy link

Maybe a separate kernel module without binding to iptables/nftables?

@aabc
Copy link
Owner

aabc commented Aug 16, 2019

What do you mean in relation to nft[ables] support?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants