fix writting new keys

yast · Aug 19, 2022 · f3ed7d3 · f3ed7d3
1 parent c0352ef
commit f3ed7d3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/lib/y2users/linux/users_writer.rb b/src/lib/y2users/linux/users_writer.rb
@@ -121,6 +121,7 @@ def write_ssh_auth_keys
             next
           end
 
+          system_user.authorized_keys = user.authorized_keys
           write_user_auth_keys(system_user, old_keys)
         end
       end
@@ -168,9 +169,9 @@ def edit_user(initial_user, target_user)
         edit_password(target_user) if initial_user.password != target_user.password
 
         previous_keys = initial_user.authorized_keys || []
-        if previous_keys != target_user.authorized_keys
-          @users_to_write_ssh_keys[target_user] = previous_keys
-        end
+        return if previous_keys == target_user.authorized_keys
+
+        @users_to_write_ssh_keys[target_user] = previous_keys
       end
 
       # Updates root aliases

diff --git a/test/lib/y2users/linux/users_writer_test.rb b/test/lib/y2users/linux/users_writer_test.rb
@@ -685,7 +685,7 @@ def issues(messages)
 
         context "and the user home exists" do
           let(:system_user) { test3.copy }
-          let(:system_config) { Y2Users::Config.new.tap { |c| c.attach([system_user]) }}
+          let(:system_config) { Y2Users::Config.new.tap { |c| c.attach([system_user]) } }
 
           before do
             allow(Yast::FileUtils).to receive(:IsDirectory).with(test3.home.path).and_return(true)