Added support for proxy_ssl_server_name to automate-es-gateway #7984
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
π© Description: What code changed, and why?
When connecting to external Opensearch clusters they may be configured in a way that requires passing along the hostname via the http request. One example of this is Opensearch hosted in Kubernetes which is using a shared ingress that depends on the hostname to be passed for internal routing to the correct service. This code updates the config api to support a new boolean value AND updates the nginx config for automate-es-gateway to check for this config value to determine whether or not to set
proxy_ssl_server_name on;
βοΈ Related Resources
π Definition of Done
You should be able to set or patch in this config:
and the nginx.conf for automate-es-gateway should include the line
proxy_ssl_server_name on;
π How to Build and Test the Change
I need help testing this as attempting to run the dev environment even on the main branch with no changes is failing for me.
β Checklist
All PRs must tick these:
With occasional exceptions, all PRs from Progress employees must tick these:
make spell
in any component directory)All PRs from Progress employees should tick these if appropriate:
Please add a note next to any checkbox above if you are NOT ticking it.
π· Screenshots, if applicable