Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package yast2-iscsi-client for openSUSE:Factory checked in at 2025-07-25 17:04:19 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/yast2-iscsi-client (Old) and /work/SRC/openSUSE:Factory/.yast2-iscsi-client.new.13279 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "yast2-iscsi-client" Fri Jul 25 17:04:19 2025 rev:151 rq:1295530 version:5.0.9 Changes: -------- --- /work/SRC/openSUSE:Factory/yast2-iscsi-client/yast2-iscsi-client.changes 2025-07-06 17:00:30.880576614 +0200 +++ /work/SRC/openSUSE:Factory/.yast2-iscsi-client.new.13279/yast2-iscsi-client.changes 2025-07-25 17:04:50.104042188 +0200 @@ -1,0 +2,12 @@ +Wed Jul 23 10:38:00 UTC 2025 - Stefan Hundhammer <shundham...@suse.com> + +- Ensure to hide passwords (bsc#1246833) +- 5.0.9 + +------------------------------------------------------------------- +Wed Jul 23 10:00:00 UTC 2025 - Knut Alejandro Anderssen González <kanders...@suse.com> + +- Do not filter netcard cards by iscsioffload feature as for example + it is not present in qede/qedi devices (bsc#1236433). + +------------------------------------------------------------------- Old: ---- yast2-iscsi-client-5.0.8.tar.bz2 New: ---- yast2-iscsi-client-5.0.9.tar.bz2 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ yast2-iscsi-client.spec ++++++ --- /var/tmp/diff_new_pack.xxmcRQ/_old 2025-07-25 17:04:50.536060114 +0200 +++ /var/tmp/diff_new_pack.xxmcRQ/_new 2025-07-25 17:04:50.540060280 +0200 @@ -17,7 +17,7 @@ Name: yast2-iscsi-client -Version: 5.0.8 +Version: 5.0.9 Release: 0 Summary: YaST2 - iSCSI Client Configuration License: GPL-2.0-only ++++++ yast2-iscsi-client-5.0.8.tar.bz2 -> yast2-iscsi-client-5.0.9.tar.bz2 ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-iscsi-client-5.0.8/package/yast2-iscsi-client.changes new/yast2-iscsi-client-5.0.9/package/yast2-iscsi-client.changes --- old/yast2-iscsi-client-5.0.8/package/yast2-iscsi-client.changes 2025-07-03 16:12:40.000000000 +0200 +++ new/yast2-iscsi-client-5.0.9/package/yast2-iscsi-client.changes 2025-07-24 13:21:09.000000000 +0200 @@ -1,4 +1,16 @@ ------------------------------------------------------------------- +Wed Jul 23 10:38:00 UTC 2025 - Stefan Hundhammer <shundham...@suse.com> + +- Ensure to hide passwords (bsc#1246833) +- 5.0.9 + +------------------------------------------------------------------- +Wed Jul 23 10:00:00 UTC 2025 - Knut Alejandro Anderssen González <kanders...@suse.com> + +- Do not filter netcard cards by iscsioffload feature as for example + it is not present in qede/qedi devices (bsc#1236433). + +------------------------------------------------------------------- Thu Jul 3 13:24:33 UTC 2025 - José Iván López González <jlo...@suse.com> - Avoid exception if discovering from a portal fails (related to diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-iscsi-client-5.0.8/package/yast2-iscsi-client.spec new/yast2-iscsi-client-5.0.9/package/yast2-iscsi-client.spec --- old/yast2-iscsi-client-5.0.8/package/yast2-iscsi-client.spec 2025-07-03 16:12:40.000000000 +0200 +++ new/yast2-iscsi-client-5.0.9/package/yast2-iscsi-client.spec 2025-07-24 13:21:09.000000000 +0200 @@ -17,7 +17,7 @@ Name: yast2-iscsi-client -Version: 5.0.8 +Version: 5.0.9 Release: 0 Summary: YaST2 - iSCSI Client Configuration License: GPL-2.0-only diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-iscsi-client-5.0.8/src/modules/IscsiClientLib.rb new/yast2-iscsi-client-5.0.9/src/modules/IscsiClientLib.rb --- old/yast2-iscsi-client-5.0.8/src/modules/IscsiClientLib.rb 2025-07-03 16:12:40.000000000 +0200 +++ new/yast2-iscsi-client-5.0.9/src/modules/IscsiClientLib.rb 2025-07-24 13:21:09.000000000 +0200 @@ -287,7 +287,7 @@ def hidePassword(orig) hidden = deep_copy(orig) hidden.each do |key, _value| - next unless key.include?("PASS") + next unless key =~ /pass/i hidden[key] = "*****" end hidden @@ -1435,7 +1435,12 @@ # # @return [Boolean] def iscsiuio_relevant? - (ISCSIUIO_MODULES & GetOffloadModules()).any? + # qedi devices could not have any network device associated with it therefore reading the + # system netcard could be wrong and in that case we will check also the configured iscsi + # ifaces (bsc#1236433). + configured = (@iface_file || {}).values.map { |i| i[:transport] }.uniq + + (ISCSIUIO_MODULES & (GetOffloadModules() | configured)).any? end # Loads the kernel modules needed to configure the iscsi client @@ -1544,7 +1549,7 @@ def potential_offload_cards # Store into hw_mods information about all the cards in the system cards = SCR.Read(path(".probe.netcard")) - hw_mods = cards.select { |c| c["iscsioffload"] }.map do |c| + hw_mods = cards.map do |c| log.info "GetOffloadItems card:#{c}" hw_mod = { "modules" => netcard_modules(c), diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-iscsi-client-5.0.8/test/iscsi_client_lib_test.rb new/yast2-iscsi-client-5.0.9/test/iscsi_client_lib_test.rb --- old/yast2-iscsi-client-5.0.8/test/iscsi_client_lib_test.rb 2025-07-03 16:12:40.000000000 +0200 +++ new/yast2-iscsi-client-5.0.9/test/iscsi_client_lib_test.rb 2025-07-24 13:21:09.000000000 +0200 @@ -1060,4 +1060,23 @@ end end end + + describe "#hidePassword" do + let(:orig) do + { + "node.name" => "my.node", + "harm.less" => "harmless", + "auth.password" => "s3cr3t", + "auth.PASSWORD" => "s3cr3t" + } + end + + it "hides passwords and only passwords" do + hidden = Yast::IscsiClientLib.hidePassword(orig) + + expect(hidden["harm.less"]).to eq("harmless") + expect(hidden["auth.PASSWORD"]).not_to include("s3cr3t") + expect(hidden["auth.password"]).not_to include("s3cr3t") + end + end end