Fix firewalld zones reader

Adapted the zones reader to the new --list-all-zones output introduced by firewalld 2.0.1 version (bsc#1216534)
yast · Oct 25, 2023 · 9d50d06 · 9d50d06
1 parent d1ae321
commit 9d50d06
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/library/network/src/lib/y2firewall/firewalld/zone_reader.rb b/library/network/src/lib/y2firewall/firewalld/zone_reader.rb
@@ -91,8 +91,8 @@ def parse_zones
       end
 
       def current_zone_from(line)
-        attribute, _value = line.split(/\s*\(active\)\s*$/)
-        zone_names.include?(attribute) ? attribute : nil
+        name, *_status = line.split
+        zone_names.include?(name) ? name : nil
       end
 
       ATTRIBUTE_MAPPING = { "summary" => "short" }.freeze

diff --git a/library/network/test/y2firewall/firewalld/zone_reader_test.rb b/library/network/test/y2firewall/firewalld/zone_reader_test.rb
@@ -43,11 +43,11 @@
         "  rich rules: ",
         "\t",
         "",
-        "public (active)",
+        "public (default, active)",
         "  summary: Public",
-        "  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: 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.",
         "  target: default",
         "  icmp-block-inversion: no",
         "  interfaces: eth0 ens3",