[pull] master from kevoreilly:master#418
Merged
pull[bot] merged 4 commits intothreatcode:masterfrom Feb 11, 2026
Merged
Conversation
* guac fix
```
✦ I have resolved the reported issues in the Guacamole web interface integration.
1. Fixed `guac-web.service` startup error: Updated web/web/asgi.py to explicitly register the http handler in ProtocolTypeRouter, complying with Channels 4.x requirements.
2. Fixed "slow and unresponsive" session: In web/guac/consumers.py, wrapped blocking GuacamoleClient methods (handshake and send) with sync_to_async. This prevents the synchronous calls from blocking the ASGI
event loop, which was causing the lag and crashes.
3. Prevented infinite loops: Updated the open method in web/guac/consumers.py to correctly handle GuacamoleClient disconnection (EOF) by breaking the receive loop and closing the WebSocket, preventing high CPU
usage on disconnect.
The files web/web/asgi.py and web/guac/consumers.py have been updated and syntax checked.
```
* Apply suggestion from @gemini-code-assist[bot]
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
* add daphne
* fix
* more fixes?
* fix
* fix
* sync
* Update consumers.py
---------
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
* ci: Update requirements.txt * Add additional Java runtime search paths to jar package The jar analysis package only searched for Oracle JRE (Java/jre*). This adds fallback paths for Microsoft OpenJDK, Eclipse Adoptium Temurin (JDK and JRE), Oracle JDK, and the deprecated OpenJDK Chocolatey package. Oracle paths remain first to preserve existing behavior. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: GitHub Actions <action@github.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )