Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow automatic port selection by tomcat #116

Closed
wants to merge 2 commits into from

Conversation

pschwarzer-tt
Copy link

@pschwarzer-tt pschwarzer-tt commented Nov 11, 2021

Short description of what this resolves:

As discussed in here: #115 here the pull request to allow automatic port selection by the arquillian tomcat plugin.

I added comments to all done changes.

Changes proposed in this pull request:

  • allow automatic port selection by Tomcat class

Fixes: #

  • changed final HTTPContext httpContext = new HTTPContext(configuration.getBindAddress(), configuration.getBindHttpPort()); to new HTTPContext(tomcat.getHost().getName(), tomcat.getConnector().getLocalPort()); in Tomcat8EmbeddedContainer.java so HTTPContext uses port choosen by Tomcat, and no longer set by user in configuration. This allows setting of 0 and to let tomcat choose an available port by its own.

  • changed ((StandardHost) host).setUnpackWARs(configuration.isUnpackArchive()); back to embeddedHostConfig.setUnpackWARs(configuration.isUnpackArchive()); because in out arquillian setup, ((StandardHost) host).setUnpackWARs(configuration.isUnpackArchive()); will not work propably

@rhusar rhusar self-assigned this Mar 12, 2024
@rhusar
Copy link
Collaborator

rhusar commented Sep 23, 2024

Hi @pschwarzer-tt, any chance you are still active and could rebase the change?

@jansohn
Copy link
Contributor

jansohn commented Oct 2, 2024

Hi @pschwarzer-tt, any chance you are still active and could rebase the change?

After looking at his profile I doubt he is still active. Can you rebase it yourself or should I open a new PR? I'm also interested in this feature / fix.

@rhusar
Copy link
Collaborator

rhusar commented Oct 2, 2024

Hi @pschwarzer-tt, any chance you are still active and could rebase the change?

After looking at his profile I doubt he is still active. Can you rebase it yourself or should I open a new PR? I'm also interested in this feature / fix.

@jansohn I am afraid so :-(

This needs more work and testing than just simple rebase, I can have a look next week at the earliest. You can take a stab if you want.

@pschwarzer-tt
Copy link
Author

Hi, its to long ago, not working with this anymore. Can someone else take over?

@rhusar
Copy link
Collaborator

rhusar commented Oct 2, 2024

Hi, its to long ago, not working with this anymore. Can someone else take over?

Thanks @pschwarzer-tt ! @jansohn has - #179

Closing!

@rhusar rhusar closed this Oct 2, 2024
@rhusar rhusar added this to the 1.2.2.Final milestone Oct 2, 2024
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.

4 participants