Skip to content

Add proxy integration test#754

Merged
Link- merged 1 commit intomainfrom
Link-/add-proxy-integration-tests
Jan 30, 2026
Merged

Add proxy integration test#754
Link- merged 1 commit intomainfrom
Link-/add-proxy-integration-tests

Conversation

@Link-
Copy link
Member

@Link- Link- commented Jan 30, 2026

Proxy Integration Test for upload-artifact

This test validates that the upload-artifact action properly honors http_proxy and https_proxy environment variables.

How it works

  1. Environment Setup: Runs in an Ubuntu container with a Squid proxy service container
  2. Traffic Blocking: Uses iptables to block all direct HTTP/HTTPS traffic (ports 80/443), allowing only traffic through the proxy
  3. Verification: Confirms direct HTTPS requests fail while proxy requests succeed
  4. Test Execution: Uploads a timestamped test artifact file

Key point

If the action ignores the proxy configuration and attempts direct connections, the upload will fail due to iptables blocking. Success indicates the action correctly routes traffic through the configured proxy.

@Link- Link- requested a review from a team as a code owner January 30, 2026 10:30
Copilot AI review requested due to automatic review settings January 30, 2026 10:30
Copy link

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

This PR adds a GitHub Actions workflow to validate that the upload-artifact action respects http_proxy and https_proxy environment variables by forcing all HTTP(S) traffic through a Squid proxy.

Changes:

  • Introduces .github/workflows/test-proxy.yml which runs a containerized Ubuntu job with a Squid proxy service.
  • Sets up iptables rules to block direct outbound HTTP/HTTPS (ports 80/443) while allowing traffic only to the proxy and DNS.
  • Verifies that direct HTTPS requests fail, HTTPS via the proxy succeeds, and then uploads a test artifact through the proxy using the local action (uses: ./).

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

@Link- Link- merged commit 47309c9 into main Jan 30, 2026
16 checks passed
Pjrich1313

This comment was marked as off-topic.

@Link- Link- deleted the Link-/add-proxy-integration-tests branch February 5, 2026 12:11
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.

6 participants