cisagov / ansible-role-guacamole

An Ansible role for installing cisagov/guacamole-composition
Creative Commons Zero v1.0 Universal
4 stars 3 forks source link

Modify Apache configuration to proxy websocket connections #11

Closed jsf9k closed 4 years ago

jsf9k commented 4 years ago

🗣 Description

This pull request:

💭 Motivation and Context

@dav3r noticed that the guacamole logs were outputting a message stating that HTTP was being used instead of websockets and performance may be degraded. He also found this document, which describes how to configure Apache to proxy guacamole. This pull request contains the Apache configuration changes from that document, which allow the guacamole websocket connections to be proxied. This results in a noticeable performance improvement.

🧪 Testing

All pre-commit hooks and molecule tests pass. The modified Apache config has been tested in env1 in our staging environment and found to greatly improve guacamole performance.

🚥 Types of Changes

✅ Checklist