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

config/network/firewall.md: needs some rework/clarifications. #512

Open
Duncaen opened this issue Sep 9, 2020 · 1 comment
Open

config/network/firewall.md: needs some rework/clarifications. #512

Duncaen opened this issue Sep 9, 2020 · 1 comment
Labels
bug Something isn't working enhancement New feature or request help wanted Extra attention is needed

Comments

@Duncaen
Copy link
Member

Duncaen commented Sep 9, 2020

We currently have here two sections "Applying rules at boot" and "Applying rules at runtime" both are ambiguous.
Both of them work at boot and at runtime, they are two alternative methods, with the one "Applying rules at boot" being clearly the better solution because they don't need a fake service and they make sure firewall rules are loaded before network services are up.

  • Change the headlines for "Applying rules at X".
  • Make clear that they are two ways of loading rules at boot and at runtime, make clear that core-services/rc.local is superior.
  • Document how to reload rules with the fake service, sv can be used and with the better core-services or rc.local method the nftables (or iptables) command can be used.
@Duncaen Duncaen added bug Something isn't working enhancement New feature or request help wanted Extra attention is needed labels Sep 9, 2020
@flexibeast
Copy link
Contributor

i'm happy to do all this once a decision is made on whether or not to split the "Firewalls" section as per #508.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants