Skip to content

Commit

Permalink
GH-43152: [Release] Require "digest/sha1" explicitly for thread safety (
Browse files Browse the repository at this point in the history
#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 <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
  • Loading branch information
kou committed Jul 5, 2024
1 parent 2de8008 commit ca7108d
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions dev/release/binary-task.rb
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
# under the License.

require "cgi/util"
require "digest/sha1"
require "digest/sha2"
require "io/console"
require "json"
Expand Down

0 comments on commit ca7108d

Please sign in to comment.