This is an automated email from the ASF dual-hosted git repository.

kou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new ca7108dcd8 GH-43152: [Release] Require "digest/sha1" explicitly for 
thread safety (#43154)
ca7108dcd8 is described below

commit ca7108dcd8c12bd29deb540f4e0c1e98fff4bcb4
Author: Sutou Kouhei <k...@clear-code.com>
AuthorDate: Sat Jul 6 06:52:17 2024 +0900

    GH-43152: [Release] Require "digest/sha1" explicitly for thread safety 
(#43154)
    
    ### Rationale for this change
    
    If we don't require `digest/sha1` explicitly, it's required automatically 
when it's needed. But it's not thread safe.
    
    ### What changes are included in this PR?
    
    Require `digest/sha1` explicitly in the main thread to avoid auto require.
    
    ### Are these changes tested?
    
    No.
    
    ### Are there any user-facing changes?
    
    No.
    * GitHub Issue: #43152
    
    Authored-by: Sutou Kouhei <k...@clear-code.com>
    Signed-off-by: Sutou Kouhei <k...@clear-code.com>
---
 dev/release/binary-task.rb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/dev/release/binary-task.rb b/dev/release/binary-task.rb
index c2386a1f52..db008b1551 100644
--- a/dev/release/binary-task.rb
+++ b/dev/release/binary-task.rb
@@ -16,6 +16,7 @@
 # under the License.
 
 require "cgi/util"
+require "digest/sha1"
 require "digest/sha2"
 require "io/console"
 require "json"

Reply via email to