Avoid zipping tempfiles as it lacks rename permissions on Windows

Fixes SeleniumHQ#9987
elgatov · Jun 27, 2022 · 28a5066 · 28a5066
1 parent 498859d
commit 28a5066
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/rb/lib/selenium/webdriver/common/zipper.rb b/rb/lib/selenium/webdriver/common/zipper.rb
@@ -72,7 +72,9 @@ def zip_file(path)
         private
 
         def with_tmp_zip(&blk)
-          Tempfile.create do |zip_path|
+          # Don't use Tempfile since it lacks rb_file_s_rename permission on Windows.
+          Dir.mktmpdir do |tmp_dir|
+            zip_path = File.join(tmp_dir, 'webdriver-zip')
             Zip::File.open(zip_path, Zip::File::CREATE, &blk)
           end
         end