Move OPENSSL_ENABLE_SHA1_SIGNATURES=1 to the test in question

[smokhov]

@w666

• I've sync'ed master with feature-disable-sha-1
• Moved setting OPENSSL_ENABLE_SHA1_SIGNATURES=1 into the sha1 test in question

This now passes on my system properly with vanilla npm run test and npm run cover.

Notice the the base branch for merge is yours feature-disable-sha-1 and not in master as it's based off #1418. Once merged, it will help to finalize #1418 and its underlying issue completely. So merge this first into #1418, and then merge that into master.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 37bd5fa1-cf05-4767-b136-1556256bf2eb

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[w666]

@smokhov I rebased #1418

[smokhov]

@smokhov I rebased #1418

Ty. Conflict fixed, ready to merge.