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

olsrd dependency on firewall3 #651

Open
champtar opened this issue Apr 2, 2021 · 3 comments
Open

olsrd dependency on firewall3 #651

champtar opened this issue Apr 2, 2021 · 3 comments

Comments

@champtar
Copy link
Member

champtar commented Apr 2, 2021

Hi @zioproto

olsrd has an implicit dependency on firewall3 for the smartgw feature.
firewall4 is based on nftables and thus part of olsrd will not work with it.
If possible use procd firewall objects (exemples https://github.com/search?p=2&q=org%3Aopenwrt+json_add_array+firewall&type=Code) else consider adding:

  • an explicit dependency on firewall3 until this is fixed
  • some checks if ip/ip6tables exists or not
@zioproto
Copy link
Contributor

zioproto commented Apr 5, 2021

@fhuberts if I remember correctly you are familiar with the smartgw feature, correct ? Could you help here ? thanks

@fhuberts
Copy link
Contributor

fhuberts commented Apr 5, 2021

It's been many years since I worked on olsrd.
But IIRC then the firewall dependend stuff is all in scripts, and those can be converted to nftables.
Might not be a trivial job but it can be done.
Not by me though ;-)

@PolynomialDivision
Copy link
Member

If possible use procd firewall objects (exemples https://github.com/search?p=2&q=org%3Aopenwrt+json_add_array+firewall&type=Code) else consider adding:

Could you do this? I think you just have to change that:
https://github.com/openwrt-routing/packages/blob/b769f1d7985a812e0b13c7d2598dc89f84e6a26e/olsrd/files/olsrd.sh#L745-L812

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

No branches or pull requests

4 participants