Skip to content

Preserve TCP and UDP bindings for the same container port#1350

Merged
cristianrgreco merged 1 commit into
mainfrom
codex/preserve-protocol-bindings
Jun 7, 2026
Merged

Preserve TCP and UDP bindings for the same container port#1350
cristianrgreco merged 1 commit into
mainfrom
codex/preserve-protocol-bindings

Conversation

@cristianrgreco

Copy link
Copy Markdown
Collaborator

Summary

  • preserve both TCP and UDP bindings when filtering the same numeric container port
  • add regression coverage for filtering both protocols together

Verification

  • npm test -- packages/testcontainers/src/utils/bound-ports.test.ts
  • npm run format
  • npm run lint
  • npm run check-compiles
  • git diff --check

Compatibility

This is a backward-compatible bug fix. Existing single-protocol filtering behavior is unchanged.

@netlify

netlify Bot commented Jun 1, 2026

Copy link
Copy Markdown

Deploy Preview for testcontainers-node ready!

Name Link
🔨 Latest commit ea11fba
🔍 Latest deploy log https://app.netlify.com/projects/testcontainers-node/deploys/6a1da82cab23720008443dbc
😎 Deploy Preview https://deploy-preview-1350--testcontainers-node.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@cristianrgreco cristianrgreco added bug Something isn't working patch Backward compatible bug fix labels Jun 1, 2026 — with ChatGPT Codex Connector
@cristianrgreco cristianrgreco merged commit 1a32238 into main Jun 7, 2026
263 checks passed
@cristianrgreco cristianrgreco deleted the codex/preserve-protocol-bindings branch June 7, 2026 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working patch Backward compatible bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant