Skip to content

Conversation

@Mcrich23
Copy link
Owner

Closes #18

With Apple container 0.3, port mapping was re-introduced to their tool. Now I am finally re-enabling it in here.

One note is that it binds to 0.0.0.0 for compatibility reasons, so be careful when exposing to the internet.

@Mcrich23 Mcrich23 requested a review from Copilot October 22, 2025 18:59
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 re-enables port binding/mapping functionality in the container compose tool, leveraging the reintroduction of port mapping in Apple container 0.3. The implementation binds to 0.0.0.0 for compatibility, with a security advisory to be cautious about internet exposure.

Key Changes:

  • Restored port mapping functionality by uncommenting and updating previously removed code
  • Updated port binding to use 0.0.0.0 prefix for compatibility
  • Minor variable mutability correction (var to let)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Mcrich23 Mcrich23 merged commit c6d661e into main Oct 22, 2025
1 check passed
@Mcrich23 Mcrich23 deleted the enable-port-binding branch October 22, 2025 19:07
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.

Support port mapping

2 participants