8000 olsrd dependency on firewall3 · Issue #651 · openwrt/routing · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

olsrd dependency on firewall3 #651

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.

Sign up for GitHub

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

Jump to bottom
Open
champtar opened this issue Apr 2, 2021 · 3 comments < 8000 /div>
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

@champtar champtar mentioned this issue Oct 6, 2021
Closed
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@fhuberts @zioproto @champtar @PolynomialDivision
0