Skip to content

Update setup perl action#5796

Merged
guhetier merged 1 commit intomainfrom
guhetier/update_setup_perl_action
Feb 17, 2026
Merged

Update setup perl action#5796
guhetier merged 1 commit intomainfrom
guhetier/update_setup_perl_action

Conversation

@guhetier
Copy link
Collaborator

Description

The build CI started to fail with errors like:

Run shogo82148/actions-setup-perl@fa3bdcb0144a50651e317f30949bb4491e74a6e4
install perl
  verifying download: D:\a\_temp\d7077245-f8e5-4d0c-bca6-93c64dc8f837
  Warning: failed to verify attestation: TypeError: Cannot read properties of null (reading 'mediaType'), continue to next attestation
  Warning: failed to verify attestation: TypeError: Cannot read properties of null (reading 'mediaType'), continue to next attestation
Error: Error: failed to verify D:\a\_temp\d7077245-f8e5-4d0c-bca6-93c64dc8f837

https://github.com/microsoft/msquic/actions/runs/22020983438/job/63904220521?pr=5794

This has been fixed in shogo82148/actions-setup-perl#2481.

Update the action to fix the CI.

Testing

CI

Documentation

N/A

@guhetier guhetier requested a review from a team as a code owner February 17, 2026 20:09
@mtfriesen
Copy link
Contributor

LGTM but we need to figure out how to not depend on shogo82148.

@codecov
Copy link

codecov bot commented Feb 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.78%. Comparing base (ed14762) to head (a4d70bc).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5796      +/-   ##
==========================================
- Coverage   85.99%   84.78%   -1.21%     
==========================================
  Files          60       60              
  Lines       18729    18729              
==========================================
- Hits        16106    15880     -226     
- Misses       2623     2849     +226     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@guhetier
Copy link
Collaborator Author

LGTM but we need to figure out how to not depend on shogo82148.

It seems to be the standard way to install perl in CI. I don't think we have a need / reason to not depend on it in our public CI.

@guhetier guhetier enabled auto-merge (squash) February 17, 2026 21:29
@mtfriesen
Copy link
Contributor

LGTM but we need to figure out how to not depend on shogo82148.

It seems to be the standard way to install perl in CI. I don't think we have a need / reason to not depend on it in our public CI.

Hmm, maybe the more pertinent but ignorant question is "why do we even use perl?" if the rest of the industry seems not to bother supporting it.

In general, we want to take dependencies with both clear support and security policies - if the action is not a first party (GitHub or Microsoft) action, then it becomes a more nuanced decision.

@guhetier
Copy link
Collaborator Author

...

Hmm, maybe the more pertinent but ignorant question is "why do we even use perl?" if the rest of the industry seems not to bother supporting it.

In general, we want to take dependencies with both clear support and security policies - if the action is not a first party (GitHub or Microsoft) action, then it becomes a more nuanced decision.

Perl is needed to build OpenSSL.

@mtfriesen
Copy link
Contributor

Hmm, maybe the more pertinent but ignorant question is "why do we even use perl?" if the rest of the industry seems not to bother supporting it.

Perl is needed to build OpenSSL.

Now it all makes sense! 😆

@guhetier guhetier merged commit e8667f0 into main Feb 17, 2026
579 of 581 checks passed
@guhetier guhetier deleted the guhetier/update_setup_perl_action branch February 17, 2026 23:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants