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

Add JkMount/JkUnmount directives to vhost #1384

Merged
merged 1 commit into from
Mar 10, 2016
Merged

Add JkMount/JkUnmount directives to vhost #1384

merged 1 commit into from
Mar 10, 2016

Conversation

smoeding
Copy link
Contributor

@smoeding smoeding commented Mar 2, 2016

mod_jk is one of the possibilities to use Apache as a frontend to Tomcat. The module understands JkMount and JkUnMount directives to configure whether an URL should be handled by Apache or by Tomcat. This patch enhances apache::vhost to allow configuration of these two directives for a virtual host.

'mod_jk' is one of the possibilities to use Apache as a frontend to
Tomcat. The module understands 'JkMount' and 'JkUnMount' directives to
configure whether an URL should be handled by Apache or by Tomcat. This
patch enhances 'apache::vhost' to allow configuration of these two
directives for a virtual host.
@igalic
Copy link
Contributor

igalic commented Mar 9, 2016

what's wrong with mod_proxy_ajp?

@smoeding
Copy link
Contributor Author

There is probably nothing wrong with it. The Tomcat Wiki has some opinions on the pros and cons of both modules.

In our case mod_jk is working pretty good and we have not seen any reason to replace it with something else.

@igalic
Copy link
Contributor

igalic commented Mar 10, 2016

imo, if you're on 2.4, you should just use mod_proxy_ajp

i for one have been using mod_proxy_http since tomcat 6…

igalic added a commit that referenced this pull request Mar 10, 2016
Add JkMount/JkUnmount directives to vhost
@igalic igalic merged commit 3f69da5 into puppetlabs:master Mar 10, 2016
@smoeding
Copy link
Contributor Author

We will check if mod_proxy_* works for us. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants