FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Provide an API to force access to the host by bsideup · Pull Request #4584 · testcontainers/testcontainers-java · GitHub

Provide an API to force access to the host - #4584

Merged
rnorth merged 2 commits into
masterfrom
force_exposed_host
Oct 18, 2021
Merged

Provide an API to force access to the host#4584
rnorth merged 2 commits into
masterfrom
force_exposed_host

Conversation

bsideup commented Oct 14, 2021

Copy link
Copy Markdown
Member

Before this change, one of the biggest limitations of host port exposing
was that the ports had to be exposed before the container starts.

This change allows forcing starting the forwarding container,
so that host listeners can be started/exposed after starting the consumer.

Before this change, one of the biggest limitations of host port exposing
was that the ports had to be exposed before the container starts.

This change allows forcing starting the forwarding container,
so that host listeners can be started/exposed after starting the consumer.
bsideup added this to the next milestone Oct 14, 2021
bsideup requested review from kiview and rnorth as code owners October 14, 2021 16:53

kiview left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Super helpful improvement for the API 👍 . We need to update the docs accordingly.

bsideup commented Oct 18, 2021

Copy link
Copy Markdown
Member Author

@kiview added a mention to the docs 👍

rnorth left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I think I'd suggest making the docs edit a bit more prominent, but this can be changed later and should not stand in the way of releasing this PR.

rnorth merged commit 81317a6 into master Oct 18, 2021
rnorth deleted the force_exposed_host branch October 18, 2021 11:15
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL