Blame SOURCES/RHEL-only-0001-Add-cockpit-by-default-to-some-zones.patch

66253d
From aaba32dd922c84662521754952e5a50198dd8625 Mon Sep 17 00:00:00 2001
66253d
From: Eric Garver <e@erig.me>
66253d
Date: Mon, 9 Jul 2018 11:29:33 -0400
66253d
Subject: [PATCH] Add cockpit by default to some zones
66253d
66253d
Fixes: #1581578
66253d
---
66253d
 config/zones/home.xml                 |  1 +
66253d
 config/zones/internal.xml             |  1 +
66253d
 config/zones/public.xml               |  1 +
66253d
 config/zones/work.xml                 |  1 +
66253d
 src/tests/features/helpers_custom.at  |  9 +++++++++
66253d
 src/tests/features/service_include.at |  2 +-
66253d
 src/tests/firewall-cmd.at             | 14 +++++++++++++-
66253d
 src/tests/regression/gh366.at         |  3 +++
66253d
 src/tests/regression/gh453.at         |  2 ++
66253d
 src/tests/regression/rhbz1514043.at   |  2 +-
66253d
 10 files changed, 33 insertions(+), 3 deletions(-)
66253d
66253d
diff --git a/config/zones/home.xml b/config/zones/home.xml
66253d
index 42b29b2f2d50..8aa8afa0e8aa 100644
66253d
--- a/config/zones/home.xml
66253d
+++ b/config/zones/home.xml
66253d
@@ -6,4 +6,5 @@
66253d
   <service name="mdns"/>
66253d
   <service name="samba-client"/>
66253d
   <service name="dhcpv6-client"/>
66253d
+  <service name="cockpit"/>
66253d
 </zone>
66253d
diff --git a/config/zones/internal.xml b/config/zones/internal.xml
66253d
index e646b48c94e8..40cb7e14424b 100644
66253d
--- a/config/zones/internal.xml
66253d
+++ b/config/zones/internal.xml
66253d
@@ -6,4 +6,5 @@
66253d
   <service name="mdns"/>
66253d
   <service name="samba-client"/>
66253d
   <service name="dhcpv6-client"/>
66253d
+  <service name="cockpit"/>
66253d
 </zone>
66253d
diff --git a/config/zones/public.xml b/config/zones/public.xml
66253d
index 49795d8c9068..617e131a4895 100644
66253d
--- a/config/zones/public.xml
66253d
+++ b/config/zones/public.xml
66253d
@@ -4,4 +4,5 @@
66253d
   <description>For use in public areas. You do not trust the other computers on networks to not harm your computer. Only selected incoming connections are accepted.</description>
66253d
   <service name="ssh"/>
66253d
   <service name="dhcpv6-client"/>
66253d
+  <service name="cockpit"/>
66253d
 </zone>
66253d
diff --git a/config/zones/work.xml b/config/zones/work.xml
66253d
index 6ea5550a40bd..9609ee6f65c2 100644
66253d
--- a/config/zones/work.xml
66253d
+++ b/config/zones/work.xml
66253d
@@ -4,4 +4,5 @@
66253d
   <description>For use in work areas. You mostly trust the other computers on networks to not harm your computer. Only selected incoming connections are accepted.</description>
66253d
   <service name="ssh"/>
66253d
   <service name="dhcpv6-client"/>
66253d
+  <service name="cockpit"/>
66253d
 </zone>
66253d
diff --git a/src/tests/features/helpers_custom.at b/src/tests/features/helpers_custom.at
66253d
index c65f067a06ec..263185c88724 100644
66253d
--- a/src/tests/features/helpers_custom.at
66253d
+++ b/src/tests/features/helpers_custom.at
66253d
@@ -17,6 +17,7 @@ NFT_LIST_RULES([inet], [filter_IN_public_allow], 0, [dnl
66253d
 		chain filter_IN_public_allow {
66253d
 			tcp dport 22 ct state new,untracked accept
66253d
 			ip6 daddr fe80::/64 udp dport 546 ct state new,untracked accept
66253d
+			tcp dport 9090 ct state new,untracked accept
66253d
 			tcp dport 2121 ct helper set "helper-ftptest-tcp"
66253d
 			tcp dport 2121 ct state new,untracked accept
66253d
 		}
66253d
@@ -27,6 +28,7 @@ IPTABLES_LIST_RULES([raw], [PRE_public_allow], 0, [dnl
66253d
 ])
66253d
 IPTABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
 	ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
+	ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
 	ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:2121 ctstate NEW,UNTRACKED
66253d
 ])
66253d
 IP6TABLES_LIST_RULES([raw], [PRE_public_allow], 0, [dnl
66253d
@@ -35,6 +37,7 @@ IP6TABLES_LIST_RULES([raw], [PRE_public_allow], 0, [dnl
66253d
 IP6TABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
 	ACCEPT tcp ::/0 ::/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
 	ACCEPT udp ::/0 fe80::/64 udp dpt:546 ctstate NEW,UNTRACKED
66253d
+	ACCEPT tcp ::/0 ::/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
 	ACCEPT tcp ::/0 ::/0 tcp dpt:2121 ctstate NEW,UNTRACKED
66253d
 ])
66253d
 
66253d
@@ -51,6 +54,7 @@ NFT_LIST_RULES([inet], [filter_IN_public_allow], 0, [dnl
66253d
 		chain filter_IN_public_allow {
66253d
 			tcp dport 22 ct state new,untracked accept
66253d
 			ip6 daddr fe80::/64 udp dport 546 ct state new,untracked accept
66253d
+			tcp dport 9090 ct state new,untracked accept
66253d
 			tcp dport 2121 ct helper set "helper-ftptest-tcp"
66253d
 			tcp dport 2121 ct state new,untracked accept
66253d
 		}
66253d
@@ -61,6 +65,7 @@ IPTABLES_LIST_RULES([raw], [PRE_public_allow], 0, [dnl
66253d
 ])
66253d
 IPTABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
 	ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
+	ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
 	ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:2121 ctstate NEW,UNTRACKED
66253d
 ])
66253d
 IP6TABLES_LIST_RULES([raw], [PRE_public_allow], 0, [dnl
66253d
@@ -69,6 +74,7 @@ IP6TABLES_LIST_RULES([raw], [PRE_public_allow], 0, [dnl
66253d
 IP6TABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
 	ACCEPT tcp ::/0 ::/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
 	ACCEPT udp ::/0 fe80::/64 udp dpt:546 ctstate NEW,UNTRACKED
66253d
+	ACCEPT tcp ::/0 ::/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
 	ACCEPT tcp ::/0 ::/0 tcp dpt:2121 ctstate NEW,UNTRACKED
66253d
 ])
66253d
 
66253d
@@ -86,6 +92,7 @@ NFT_LIST_RULES([inet], [filter_IN_public_allow], 0, [dnl
66253d
 		chain filter_IN_public_allow {
66253d
 			tcp dport 22 ct state new,untracked accept
66253d
 			ip6 daddr fe80::/64 udp dport 546 ct state new,untracked accept
66253d
+			tcp dport 9090 ct state new,untracked accept
66253d
 			tcp dport 21 ct helper set "helper-ftp-tcp"
66253d
 			tcp dport 2121 ct helper set "helper-ftptest-tcp"
66253d
 			tcp dport 2121 ct state new,untracked accept
66253d
@@ -99,6 +106,7 @@ IPTABLES_LIST_RULES([raw], [PRE_public_allow], 0, [dnl
66253d
 ])
66253d
 IPTABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
 	ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
+	ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
 	ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:2121 ctstate NEW,UNTRACKED
66253d
 	ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:21 ctstate NEW,UNTRACKED
66253d
 ])
66253d
@@ -109,6 +117,7 @@ IP6TABLES_LIST_RULES([raw], [PRE_public_allow], 0, [dnl
66253d
 IP6TABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
 	ACCEPT tcp ::/0 ::/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
 	ACCEPT udp ::/0 fe80::/64 udp dpt:546 ctstate NEW,UNTRACKED
66253d
+	ACCEPT tcp ::/0 ::/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
 	ACCEPT tcp ::/0 ::/0 tcp dpt:2121 ctstate NEW,UNTRACKED
66253d
 	ACCEPT tcp ::/0 ::/0 tcp dpt:21 ctstate NEW,UNTRACKED
66253d
 ])
66253d
diff --git a/src/tests/features/service_include.at b/src/tests/features/service_include.at
66253d
index 219d5b42767b..0bf59f63b81b 100644
66253d
--- a/src/tests/features/service_include.at
66253d
+++ b/src/tests/features/service_include.at
66253d
@@ -117,7 +117,7 @@ FWD_CHECK([--zone=drop --list-services], 0, [dnl
66253d
 
66253d
 ])
66253d
 FWD_CHECK([--zone=public --list-services], 0, [dnl
66253d
-dhcpv6-client ssh
66253d
+cockpit dhcpv6-client ssh
66253d
 ])
66253d
 FWD_CHECK([-q --permanent --service=my-service-with-include --remove-include=does-not-exist])
66253d
 FWD_RELOAD
66253d
diff --git a/src/tests/firewall-cmd.at b/src/tests/firewall-cmd.at
66253d
index 0e0d3938da0a..540bdb8b1065 100644
66253d
--- a/src/tests/firewall-cmd.at
66253d
+++ b/src/tests/firewall-cmd.at
66253d
@@ -1144,6 +1144,7 @@ FWD_START_TEST([rich rules priority])
66253d
         chain filter_IN_public_allow {
66253d
         tcp dport 22 ct state new,untracked accept
66253d
         ip6 daddr fe80::/64 udp dport 546 ct state new,untracked accept
66253d
+        tcp dport 9090 ct state new,untracked accept
66253d
         tcp dport 1122 ct state new,untracked accept
66253d
         tcp dport 3333 ct state new,untracked accept
66253d
         tcp dport 4444 ct state new,untracked accept
66253d
@@ -1159,6 +1160,7 @@ FWD_START_TEST([rich rules priority])
66253d
     ])
66253d
     IPTABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
         ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
+        ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
         ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:1122 ctstate NEW,UNTRACKED
66253d
         ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:3333 ctstate NEW,UNTRACKED
66253d
         ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:4444 ctstate NEW,UNTRACKED
66253d
@@ -1173,6 +1175,7 @@ FWD_START_TEST([rich rules priority])
66253d
     IP6TABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
         ACCEPT tcp ::/0 ::/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
         ACCEPT udp ::/0 fe80::/64 udp dpt:546 ctstate NEW,UNTRACKED
66253d
+        ACCEPT tcp ::/0 ::/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
         ACCEPT tcp ::/0 ::/0 tcp dpt:1122 ctstate NEW,UNTRACKED
66253d
         ACCEPT tcp ::/0 ::/0 tcp dpt:3333 ctstate NEW,UNTRACKED
66253d
         ACCEPT tcp ::/0 ::/0 tcp dpt:4444 ctstate NEW,UNTRACKED
66253d
@@ -1254,6 +1257,7 @@ FWD_START_TEST([rich rules priority])
66253d
         chain filter_IN_public_allow {
66253d
         tcp dport 22 ct state new,untracked accept
66253d
         ip6 daddr fe80::/64 udp dport 546 ct state new,untracked accept
66253d
+        tcp dport 9090 ct state new,untracked accept
66253d
         }
66253d
         }
66253d
     ])
66253d
@@ -1357,6 +1361,7 @@ FWD_START_TEST([rich rules priority])
66253d
     ])
66253d
     IPTABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
         ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
+        ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
     ])
66253d
     IPTABLES_LIST_RULES([filter], [FWDI_public_pre], 0, [dnl
66253d
     ])
66253d
@@ -1391,6 +1396,7 @@ FWD_START_TEST([rich rules priority])
66253d
     IP6TABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
         ACCEPT tcp ::/0 ::/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
         ACCEPT udp ::/0 fe80::/64 udp dpt:546 ctstate NEW,UNTRACKED
66253d
+        ACCEPT tcp ::/0 ::/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
     ])
66253d
     IP6TABLES_LIST_RULES([filter], [FWDI_public_pre], 0, [dnl
66253d
     ])
66253d
@@ -1438,6 +1444,7 @@ FWD_START_TEST([rich rules priority])
66253d
         chain filter_IN_public_allow {
66253d
         tcp dport 22 ct state new,untracked accept
66253d
         ip6 daddr fe80::/64 udp dport 546 ct state new,untracked accept
66253d
+        tcp dport 9090 ct state new,untracked accept
66253d
         icmp type echo-request accept
66253d
         icmpv6 type echo-request accept
66253d
         }
66253d
@@ -1478,6 +1485,7 @@ FWD_START_TEST([rich rules priority])
66253d
     ])
66253d
     IPTABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
         ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
+        ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
         ACCEPT icmp -- 0.0.0.0/0 0.0.0.0/0 icmptype 8
66253d
     ])
66253d
     IPTABLES_LIST_RULES([filter], [FWDI_public_pre], 0, [dnl
66253d
@@ -1500,6 +1508,7 @@ FWD_START_TEST([rich rules priority])
66253d
     IP6TABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
         ACCEPT tcp ::/0 ::/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
         ACCEPT udp ::/0 fe80::/64 udp dpt:546 ctstate NEW,UNTRACKED
66253d
+        ACCEPT tcp ::/0 ::/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
         ACCEPT icmpv6 ::/0 ::/0 ipv6-icmptype 128
66253d
     ])
66253d
     IP6TABLES_LIST_RULES([filter], [FWDI_public_pre], 0, [dnl
66253d
@@ -1556,6 +1565,7 @@ FWD_START_TEST([rich rules priority])
66253d
         chain filter_IN_public_allow {
66253d
         tcp dport 22 ct state new,untracked accept
66253d
         ip6 daddr fe80::/64 udp dport 546 ct state new,untracked accept
66253d
+        tcp dport 9090 ct state new,untracked accept
66253d
         }
66253d
         }
66253d
     ])
66253d
@@ -1593,6 +1603,7 @@ FWD_START_TEST([rich rules priority])
66253d
     ])
66253d
     IPTABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
         ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
+        ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
     ])
66253d
     IPTABLES_LIST_RULES([filter], [IN_public_deny], 0, [dnl
66253d
     ])
66253d
@@ -1613,6 +1624,7 @@ FWD_START_TEST([rich rules priority])
66253d
     IP6TABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
         ACCEPT tcp ::/0 ::/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
         ACCEPT udp ::/0 fe80::/64 udp dpt:546 ctstate NEW,UNTRACKED
66253d
+        ACCEPT tcp ::/0 ::/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
     ])
66253d
     IP6TABLES_LIST_RULES([filter], [IN_public_deny], 0, [dnl
66253d
     ])
66253d
@@ -1638,7 +1650,7 @@ FWD_START_TEST([rich rules priority])
66253d
         icmp-block-inversion: no
66253d
         interfaces:
66253d
         sources:
66253d
-        services: dhcpv6-client ssh
66253d
+        services: cockpit dhcpv6-client ssh
66253d
         ports:
66253d
         protocols:
66253d
         masquerade: no
66253d
diff --git a/src/tests/regression/gh366.at b/src/tests/regression/gh366.at
66253d
index 1441a6be53bf..51ff504e6a9d 100644
66253d
--- a/src/tests/regression/gh366.at
66253d
+++ b/src/tests/regression/gh366.at
66253d
@@ -7,6 +7,7 @@ table inet firewalld {
66253d
 chain filter_IN_public_allow {
66253d
 tcp dport 22 ct state new,untracked accept
66253d
 ip6 daddr fe80::/64 udp dport 546 ct state new,untracked accept
66253d
+tcp dport 9090 ct state new,untracked accept
66253d
 ip daddr 224.0.0.251 udp dport 5353 ct state new,untracked accept
66253d
 ip6 daddr ff02::fb udp dport 5353 ct state new,untracked accept
66253d
 }
66253d
@@ -14,11 +15,13 @@ ip6 daddr ff02::fb udp dport 5353 ct state new,untracked accept
66253d
 ])
66253d
 IPTABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
 ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
+ACCEPT tcp -- 0.0.0.0/0 0.0.0.0/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
 ACCEPT udp -- 0.0.0.0/0 224.0.0.251 udp dpt:5353 ctstate NEW,UNTRACKED
66253d
 ])
66253d
 IP6TABLES_LIST_RULES([filter], [IN_public_allow], 0, [dnl
66253d
 ACCEPT tcp ::/0 ::/0 tcp dpt:22 ctstate NEW,UNTRACKED
66253d
 ACCEPT udp ::/0 fe80::/64 udp dpt:546 ctstate NEW,UNTRACKED
66253d
+ACCEPT tcp ::/0 ::/0 tcp dpt:9090 ctstate NEW,UNTRACKED
66253d
 ACCEPT udp ::/0 ff02::fb udp dpt:5353 ctstate NEW,UNTRACKED
66253d
 ])])
66253d
 
66253d
diff --git a/src/tests/regression/gh453.at b/src/tests/regression/gh453.at
66253d
index f57a79dcf9a2..6d820fce840a 100644
66253d
--- a/src/tests/regression/gh453.at
66253d
+++ b/src/tests/regression/gh453.at
66253d
@@ -18,6 +18,7 @@ NFT_LIST_RULES([inet], [filter_IN_public_allow], 0, [dnl
66253d
 	chain filter_IN_public_allow {
66253d
 	tcp dport 22 ct state new,untracked accept
66253d
 	ip6 daddr fe80::/64 udp dport 546 ct state new,untracked accept
66253d
+	tcp dport 9090 ct state new,untracked accept
66253d
 	tcp dport 21 ct helper set "helper-ftp-tcp"
66253d
 	tcp dport 21 ct state new,untracked accept
66253d
 	}
66253d
@@ -42,6 +43,7 @@ NFT_LIST_RULES([inet], [filter_IN_public_allow], 0, [dnl
66253d
 	chain filter_IN_public_allow {
66253d
 	tcp dport 22 ct state new,untracked accept
66253d
 	ip6 daddr fe80::/64 udp dport 546 ct state new,untracked accept
66253d
+	tcp dport 9090 ct state new,untracked accept
66253d
 	tcp dport 21 ct helper set "helper-ftp-tcp"
66253d
 	tcp dport 21 ct state new,untracked accept
66253d
 	tcp dport 5060 ct helper set "helper-sip-tcp"
66253d
diff --git a/src/tests/regression/rhbz1514043.at b/src/tests/regression/rhbz1514043.at
66253d
index efc33e09478b..241cf547f7f3 100644
66253d
--- a/src/tests/regression/rhbz1514043.at
66253d
+++ b/src/tests/regression/rhbz1514043.at
66253d
@@ -5,7 +5,7 @@ FWD_CHECK([-q --set-log-denied=all])
66253d
 FWD_CHECK([-q --permanent --zone=public --add-service=samba])
66253d
 FWD_RELOAD
66253d
 FWD_CHECK([--zone=public --list-all | TRIM | grep ^services], 0, [dnl
66253d
-services: dhcpv6-client samba ssh
66253d
+services: cockpit dhcpv6-client samba ssh
66253d
 ])
66253d
 
66253d
 dnl check that log denied actually took effect
66253d
-- 
66253d
2.23.0
66253d