Skip to content

Conversation

@johan-carlsson
Copy link
Contributor

Add support for configuring Docker Compose and test ports via environment variables with defaults matching the existing hardcoded values:

  • TINYPROXY_PORT (default: 8888)
  • TINYPROXY_AUTH_PORT (default: 8889)
  • HTTPBIN_HTTP_PORT (default: 8080)
  • HTTPBIN_HTTPS_PORT (default: 8443)

This allows running tests when default ports are already in use.

Add support for configuring Docker Compose and test ports via environment
variables with defaults matching the existing hardcoded values:

- TINYPROXY_PORT (default: 8888)
- TINYPROXY_AUTH_PORT (default: 8889)
- HTTPBIN_HTTP_PORT (default: 8080)
- HTTPBIN_HTTPS_PORT (default: 8443)

This allows running tests when default ports are already in use.
Copy link
Contributor

@whatyouhide whatyouhide left a comment

Choose a reason for hiding this comment

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

Lovely, thank you @johan-carlsson 🫶

@coveralls
Copy link

Pull Request Test Coverage Report for Build 7805ac207897a323d9e4d74b7664cbf3abde2dd0-PR-475

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 87.772%

Totals Coverage Status
Change from base Build e28c85aad15d1f0cfcb1d5e4f4abada5f37f0f11: 0.0%
Covered Lines: 1292
Relevant Lines: 1472

💛 - Coveralls

@whatyouhide whatyouhide merged commit 2245192 into elixir-mint:main Jan 29, 2026
3 checks passed
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