nftables: allow 169.254.169.0/24 and set log level to notice

[chef.git] / cookbooks / networking / templates / default / nftables.conf.erb

diff --git a/cookbooks/networking/templates/default/nftables.conf.erb b/cookbooks/networking/templates/default/nftables.conf.erb
index 7fc46cff29896c7045b03d83699763d12b786650..1ef38da7f544d85b39a4453a602d889ac0d5466f 100644 (file)
--- a/cookbooks/networking/templates/default/nftables.conf.erb
+++ b/cookbooks/networking/templates/default/nftables.conf.erb
@@ -1,35 +1,37 @@
 #!/usr/sbin/nft -f
 
-<%- unless @interfaces.empty? %>
+<% unless @interfaces.empty? -%>
 define external-interfaces = { <%= @interfaces.sort.uniq.join(", ") %> }
-<%- end %>
-
-define ip-private-addresses = { 0.0.0.0, 10.0.0.0/8, 127.0.0.0/8, 169.254.0.0/16, 172.16.0.0/12, 192.0.2.0/24, 192.168.0.0/16, 224.0.0.0/4 }
-define ip6-private-addresses = { 2001:db8::/32, fc00::/7, ff00::/8 }
-
-table inet filter {
+<% end -%>
+# Exclude 169.254.169.0/24 from ip-private-addresses as is widely by cloud providers.
+define ip-private-addresses = { 0.0.0.0, 10.0.0.0/8, 127.0.0.0/8, 169.254.0.0-169.254.168.255, 169.254.170.0-169.254.255.255, 172.16.0.0/12, 192.0.2.0/24, 192.168.0.0/16 }
+define ip-multicast-addresses = { 224.0.0.0/4 }
+define ip6-private-addresses = { 2001:db8::/32, fc00::/7 }
+define ip6-multicast-addresses = { ff00::/8 }
+
+table inet chef-filter {
   set ip-osm-addresses {
     type ipv4_addr
-<%- unless Array(@hosts["inet"]).empty? %>
-    elements = { <%= Array(@hosts["inet"]).sort.join(", ") %> }
-<%- end %>
+<% unless Array(@hosts[:inet]).empty? -%>
+    elements = { <%= Array(@hosts[:inet]).sort.join(", ") %> }
+<% end -%>
   }
 
   set ip6-osm-addresses {
     type ipv6_addr
-<%- unless Array(@hosts["inet"]).empty? %>
-    elements = { <%= Array(@hosts["inet6"]).sort.join(", ") %> }
-<%- end %>
+<% unless Array(@hosts[:inet6]).empty? -%>
+    elements = { <%= Array(@hosts[:inet6]).sort.join(", ") %> }
+<% end -%>
   }
 
-  set ip-blacklist {
+  set ip-blocklist {
     type ipv4_addr
-    flags dynamic
+    flags interval
   }
 
-  set ip6-blacklist {
+  set ip6-blocklist {
     type ipv6_addr
-    flags dynamic
+    flags interval
   }
 
   set ratelimit-icmp-echo-ip {
@@ -44,44 +46,46 @@ table inet filter {
     timeout 120s
   }
 
-<%- node[:networking][:firewall][:sets].each do |set| %>
-  set <%= set %> {
-<%- if set.end_with?("-ip") %>
-    type ipv4_addr
-<%- elsif set.end_with?("-ip6") %>
-    type ipv6_addr
-<%- end %>
-    flags dynamic
-<%- unless set.start_with?("connlimit-") %>
-    timeout 120s
-<%- end %>
+<% node[:networking][:firewall][:sets].each do |set| -%>
+  set <%= set[:name] %> {
+    type <%= set[:type] %>
+<% if set[:flags] -%>
+    flags <%= set[:flags].join(", ") %>
+<% end -%>
+<% if set[:timeout] -%>
+    timeout <%= set[:timeout] %>s
+<% end -%>
   }
 
-<%- end %>
+<% end -%>
+
+<% node[:networking][:firewall][:helpers].each do |helper| -%>
+  ct helper <%= helper[:name] %> {
+    type "<%= helper[:helper] %>" protocol <%= helper[:protocol] %>
+  }
+
+<% end -%>
   chain log-and-drop {
-    limit rate 1/second log
+    limit rate 1/second log level notice
     drop
   }
 
   chain log-and-reject {
-    limit rate 1/second log
+    limit rate 1/second log level notice
     reject
   }
 
   chain incoming {
-<%- if node[:networking][:firewall][:whitelist].empty? %>
-    ip saddr { $ip-private-addresses } jump log-and-drop
-<%- else %>
-    ip saddr { $ip-private-addresses } ip saddr != { <%= node[:networking][:firewall][:whitelist].sort.join(", ") %> } jump log-and-drop
-<%- end %>
-    ip6 saddr { $ip6-private-addresses } jump log-and-drop
+<% if node[:networking][:firewall][:allowlist].empty? -%>
+    ip saddr { $ip-private-addresses, $ip-multicast-addresses } jump log-and-drop
+<% else -%>
+    ip saddr { $ip-private-addresses, $ip-multicast-addresses } ip saddr != { <%= node[:networking][:firewall][:allowlist].sort.join(", ") %> } jump log-and-drop
+<% end -%>
+    ip6 saddr { $ip6-private-addresses, $ip6-multicast-addresses } jump log-and-drop
 
-    ip saddr @ip-blacklist jump log-and-drop
-    ip6 saddr @ip6-blacklist jump log-and-drop
+    ip saddr @ip-blocklist jump log-and-drop
+    ip6 saddr @ip6-blocklist jump log-and-drop
 
-    ct state { established, related } accept
-
-    icmp type { destination-unreachable } accept
     icmp type { echo-request } update @ratelimit-icmp-echo-ip { ip saddr limit rate 1/second } accept
     icmp type { echo-request } drop
 
@@ -89,6 +93,8 @@ table inet filter {
     icmpv6 type { echo-request } update @ratelimit-icmp-echo-ip6 { ip6 saddr limit rate 1/second } accept
     icmpv6 type { echo-request } drop
 
+    ct state { established, related } accept
+
     meta l4proto { icmp, icmpv6 } jump log-and-drop
 
     tcp flags & (fin|syn|rst|psh|ack|urg) == fin|psh|urg jump log-and-drop
@@ -99,24 +105,24 @@ table inet filter {
     tcp flags & (fin|psh|ack) == fin|psh jump log-and-drop
     tcp sport 0 tcp flags & (fin|syn|rst|ack) == syn jump log-and-drop
 
-<%- node[:networking][:firewall][:incoming].uniq.each do |rule| %>
+<% node[:networking][:firewall][:incoming].uniq.each do |rule| -%>
     <%= rule %>
-<%- end %>
+<% end -%>
 
     jump log-and-drop
   }
 
   chain outgoing {
-<%- if node[:networking][:firewall][:whitelist].empty? %>
+<% if node[:networking][:firewall][:allowlist].empty? -%>
     ip daddr { $ip-private-addresses } jump log-and-drop
-<%- else %>
-    ip daddr { $ip-private-addresses } ip daddr != { <%= node[:networking][:firewall][:whitelist].sort.join(", ") %> } jump log-and-drop
-<%- end %>
+<% else -%>
+    ip daddr { $ip-private-addresses } ip daddr != { <%= node[:networking][:firewall][:allowlist].sort.join(", ") %> } jump log-and-drop
+<% end -%>
     ip6 daddr { $ip6-private-addresses } jump log-and-drop
 
-<%- node[:networking][:firewall][:outgoing].each do |rule| %>
+<% node[:networking][:firewall][:outgoing].each do |rule| -%>
     <%= rule %>
-<%- end %>
+<% end -%>
 
     accept
   }
@@ -124,9 +130,9 @@ table inet filter {
   chain input {
     type filter hook input priority filter;
 
-<%- unless @interfaces.empty? %>
+<% unless @interfaces.empty? -%>
     iifname { $external-interfaces } jump incoming
-<%- end %>
+<% end -%>
 
     accept
   }
@@ -134,10 +140,10 @@ table inet filter {
   chain forward {
     type filter hook forward priority filter;
 
-<%- unless @interfaces.empty? %>
+<% unless @interfaces.empty? -%>
     iifname { $external-interfaces } jump incoming
     oifname { $external-interfaces } jump outgoing
-<%- end %>
+<% end -%>
 
     accept
   }
@@ -145,24 +151,24 @@ table inet filter {
   chain output {
     type filter hook output priority filter;
 
-<%- unless @interfaces.empty? %>
+<% unless @interfaces.empty? -%>
     oifname { $external-interfaces } jump outgoing
-<%- end %>
+<% end -%>
 
     accept
   }
 }
-<%- if node[:roles].include?("gateway") %>
+<% if node[:roles].include?("gateway") -%>
 
-table ip nat {
+table ip chef-nat {
   chain postrouting {
     type nat hook postrouting priority srcnat;
 
-<%- node.interfaces(:role => :external, :family => :inet).each do |external| %>
-<%- node.interfaces(:role => :internal, :family => :inet).each do |internal| %>
-    oifname { <%= external[:interface] %> } ip saddr { <%= internal[:network] %>/<%= internal[:prefix] %> } snat <%= external[:address] %>
-<%- end %>
-<%- end %>
+<% node.interfaces(:role => :external).each do |external| -%>
+<% node.ipaddresses(:role => :internal, :family => :inet).each do |internal| -%>
+    oifname { <%= external[:interface] %> } ip saddr { <%= internal.subnet %> } snat <%= external[:inet][:address] %>
+<% end -%>
+<% end -%>
   }
 }
-<%- end %>
+<% end -%>