nftableslib icon indicating copy to clipboard operation
nftableslib copied to clipboard

L2Rule Support

Open Adphi opened this issue 4 years ago • 2 comments
trafficstars

Hello,

First of all, thanks for you amazing work.

I can see that 2 methods are defined, but neither exported, or used:

func inputIntfByName(intf string) []expr.Any
func outputIntfByName(intf string) []expr.Any

Are there any plan to support L2Rule as there already is L3Rule and L4Rule ?

Best regards.

Adphi avatar Dec 20 '20 19:12 Adphi

Hello, you are right, these are some leftovers I was using at early stages of the development. I will need to do a bit of house keeping. With regards to your L2 rule question, my primary goal was l3/l4. Currently I do not have plans to add L2, I see 2 possibilities, either you contribute L2 Rule implementation or describe your use case in greater details and I will try to add it but no guarantee on ETA.

sbezverk avatar Dec 22 '20 03:12 sbezverk

Thanks for your answer.

I am currently working on a small pure go - API first (grpc/rest) router and may need to provide L2 Rules support, but there is no rush, it could be in a later release. I did plan to take a deeper look at the nftables expressions logic, but, as for now I didn't find the time to do it. But as soon as I do, I'd be glad to contribute.

Adphi avatar Dec 22 '20 16:12 Adphi