Reverse proxy should be optional

[onknows]

In most cases a reverse proxy is not needed. If a reverse proxy is needed more than likely it will be configured in a separate cookbook.

Typically you don't create a reverse proxy per product. You would use reverse proxy for JIRA, Confluence and possibly other products. But you won't want a proxy for JIRA, one for Confluence.

IMHO the cookbook will be more useful if it possible to exclude the reverse proxy. Or better, if it is optional.

For testing purposes it may be useful to have reverse proxy as a option I'm not sure.

[legal90]

@onknows Thank you for proposal. I agree that sometimes it could be desirable to configure reverse proxy outside of this application cookbook, so it makes sense.

However, at the moment I can't provide a good solution for that, but we are open for pull-requests :)

[mrjcleaver]

How about a flag such that https://github.com/parallels-cookbooks/confluence/blob/master/recipes/default.rb#L26 only runs if the flag to use a proxy is set?

[mrjcleaver]

?

[onknows]

Sure, that does it.

[mrjcleaver]

@patcon FYI

[et304383]

Anything that can be done to move this along folks? We'd like installing apache2 to be optional as well based on a flag like the JIRA cookbook does it:

https://github.com/afklm/jira/blob/master/recipes/default.rb#L8

[damacus]

Closing due to inactivity.

If this is still an issue please reopen or open another issue. Alternatively drop by the #sous-chefs channel on the Chef Community Slack and we'll be happy to help!

Thanks, Sous-Chefs

[github-actions[bot]]

Marking stale due to inactivity. Remove stale label or comment or this will be closed in 7 days. Alternatively drop by the #sous-chefs channel on the Chef Community Slack and we'll be happy to help! Thanks, Sous-Chefs.