On Tue, Mar 19, 2019 at 04:48:11PM -0600, Tim Flink wrote: > The current template assumes that websockets are at the base of a URL > but that is not true for our buildmaster. This patch adds remotepath > to the end of the websocket url if remotepath is defined. > --- > roles/httpd/reverseproxy/templates/reversepassproxy.conf | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/roles/httpd/reverseproxy/templates/reversepassproxy.conf > b/roles/httpd/reverseproxy/templates/reversepassproxy.conf index > deec40b..6131b5d 100644 --- > a/roles/httpd/reverseproxy/templates/reversepassproxy.conf +++ > b/roles/httpd/reverseproxy/templates/reversepassproxy.conf @@ -29,10 > +29,18 @@ SSLProxyEngine On <Proxy > "balancer://{{balancer_name}}-websocket"> {% for member in > balancer_members %} {% if > http_not_https_yes_this_is_insecure_and_i_feel_bad %} > + {% if remotepath is defined %} > + BalancerMember "ws://{{ member }}{{ remotepath }} > + {% else %} > BalancerMember "ws://{{ member }}" > + {% endif %} > + {% else %} > + {% if remotepath is defined %} > + BalancerMember "ws://{{ member }}{{ remotepath }} You're missing on 's' here: wss:// (the first if is about unsecure: ws, we're in the else where about secure ws, so wss). > {% else %} > BalancerMember "wss://{{ member }}" > {% endif %} > + {% endif %} > {% endfor %} > </Proxy> One typo, otherwise +1 for me Pierre _______________________________________________ infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx