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

[Suggestion] - Add variables for application tunnels to values.yaml #659

Open
1 task done
BryanGuffeySonos opened this issue Aug 31, 2023 · 1 comment
Open
1 task done
Labels
enhancement New feature or request

Comments

@BryanGuffeySonos
Copy link

Suggestion

Many consumers of the DC helm charts will need to connect to Atlassian Cloud application tunnels because they are running one application in Cloud and one on-prem using DC.

In order to successfully deploy application tunnels, the server.xml file needs to be modified with a new HTTP connector and a new JVM argument needs to be added, specifying an upstream port. See here: https://support.atlassian.com/organization-administration/docs/configure-required-connections-and-upstream-ports/

Additionally, one can restrict all traffic to localhost with additional modifications to server.xml, see here: https://support.atlassian.com/organization-administration/docs/configure-required-connections-and-upstream-ports/#Optional--Restrict-access-to-the-HTTP-connector-to-localhost

Finally, HTTPS redirection must be removed from the web.xml file, see here: https://support.atlassian.com/organization-administration/docs/configure-required-connections-and-upstream-ports/#Remove-HTTPS-redirection-from-the-web.xml-file

This seems like an excellent place to add some variables such as APPLICATION_TUNNEL_NEEDED, APPLICATION_TUNNEL_CONNECTOR_PORT, and RESTRICT_APPLICATION_TUNNEL_TO_LOCALHOST` which could then be used to generate the appropriate connector in server.xml, adjust the Engine if necessary, and add the flag to the JVM arguments, along with removing HTTPS redirection from web.xml, all from within the helm chart.

Product

Jira

Code of Conduct

  • I agree to follow this project's Code of Conduct
@bianchi2
Copy link
Collaborator

@BryanGuffeySonos indeed, there have been a few cases involving app tunnels. We'll definitely take a look at it, thanks for raising it!

@bianchi2 bianchi2 added the enhancement New feature or request label Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants