Feature #8712
openfirewall: add default app policy options
Description
Add options to specific a default policy for policies that aren't specified.
firewall:
policies:
default-policy:
- "accept:hook"
http:
default-policy:
- "accept:hook"
request-started:
- "accept:hook"
request-trailer:
- "accept:hook"
request-complete:
- "accept:hook"
# implied request-body uses firewall.policies.http.default-policy
# implied tls uses firewall.policies.default-policy
VJ Updated by Victor Julien about 1 month ago
- Related to Feature #7701: firewall: configurable default policies added
- Related to Bug #8389: firewall: Add monitor/observe mode to preview firewall policy without enforcing drops added
VJ Updated by Victor Julien about 1 month ago
- Subject changed from firewall: add default app policy to firewall: add default app policy options
LS Updated by Lukas Sismis about 1 month ago
- Status changed from New to Assigned
- Assignee set to Lukas Sismis
- Target version changed from TBD to 9.0.0-beta1
LS Updated by Lukas Sismis about 1 month ago
Would it be ok to redefine the policy section of the configuration file to a format like this?
The current format as is in: https://github.com/OISF/suricata/blob/43bc2db41e4f7356f6cadb3247750d66e27ea8fe/suricata.yaml.in#L2396
Feels ambiguous, especially since it doesn't allow defining policies to packet-related tables only (hence adding a "packet" node).
The same would be for defining global app policy option (hence adding "app" configuration node)
firewall:
policies:
default-policy: ["accept:hook"] # global fallback (all hooks)
packet:
default-policy: ["drop:packet"] # fallback for packet hooks
filter: ["drop:packet"]
pre-flow: ["accept:hook"]
pre-stream: ["accept:hook"]
app:
default-policy: ["drop:flow"] # fallback for all app hooks
dns:
default-policy: ["drop:flow"] # fallback for dns hooks
request-started: ["accept:hook"]
request-complete: ["drop:flow", "alert"]
response-started: ["accept:tx"]
VJ Updated by Victor Julien about 1 month ago
I think it makes sense. @Yash Datre any opinions?
LS Updated by Lukas Sismis about 1 month ago
Not particularly related to this ticket, but I would like to propose the change of the app layer policy from "http" to "http1" to be in line with FW rules.
YD Updated by Yash Datre about 1 month ago
Yes, this should solve the use-case.
AP Updated by Aneesh Patel 22 days ago
Any update on this ticket? This is a pretty critical one for us
LS Updated by Lukas Sismis 22 days ago
Hello, yes, it is on top of my list. I have it implemented, and after my testing/review, I expect I'll create a PR on Monday/Tuesday.
LS Updated by Lukas Sismis 17 days ago
- Status changed from In Progress to In Review
LS Updated by Lukas Sismis 17 days ago
- Label Needs backport to 8.0 added
OT Updated by OISF Ticketbot 17 days ago
- Subtask #8770 added
OT Updated by OISF Ticketbot 17 days ago
- Label deleted (
Needs backport to 8.0)
JF Updated by Juliana Fajardini Reichow 12 days ago
- Related to Feature #8781: firewall: allow bypass as a default policy added