Skip to content

tls fix#19

Open
rlerdorf wants to merge 2 commits intomainfrom
fix/invoke-webrequest-tls
Open

tls fix#19
rlerdorf wants to merge 2 commits intomainfrom
fix/invoke-webrequest-tls

Conversation

@rlerdorf
Copy link
Copy Markdown
Member

@rlerdorf rlerdorf commented Apr 3, 2026

  • Adds the same TLS 1.2/1.3 enforcement that was already applied to determine-revision.ps1 in v0.12
  • Adds -UseBasicParsing to all 5 Invoke-WebRequest calls.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the action’s PowerShell runner script to enforce modern TLS protocols and make Invoke-WebRequest calls compatible with older Windows PowerShell environments.

Changes:

  • Enforce TLS 1.2/1.3 via ServicePointManager.SecurityProtocol.
  • Add -UseBasicParsing to all Invoke-WebRequest calls (5 total) to avoid IE/DOM dependencies in Windows PowerShell.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rlerdorf rlerdorf requested a review from cmb69 April 3, 2026 09:38
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.

2 participants