Skip to content

client: add gui_rpc_msg_debug log flag - #7151

Merged
AenBleidd merged 3 commits into
masterfrom
dpa_gui_rpc4
Jun 24, 2026
Merged

client: add gui_rpc_msg_debug log flag#7151
AenBleidd merged 3 commits into
masterfrom
dpa_gui_rpc4

Conversation

@davidpanderson

@davidpanderson davidpanderson commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

The new flag prints the request and reply messages (verbose).
gui_rpc_debug just prints connection info and RPC commands.


Summary by cubic

Adds a new gui_rpc_msg_debug log flag to print full GUI RPC request and reply messages. Keeps gui_rpc_debug focused on connection-level events and command tags, with clearer socket IDs and error logs.

  • New Features

    • Added gui_rpc_msg_debug flag; logs full GUI RPC requests/replies.
    • Expanded gui_rpc_debug logs with socket IDs, connection/accept/handler errors, and request tag.
    • Wired flag into cc_config.xml parse/write.
  • Refactors

    • Removed unused send_quits() and quits_sent() from GUI RPC server.
    • Tweaked Manager UI text: title now “event log options”; heading mentions “messages” instead of “diagnostic messages”.

Written for commit b9d7e51. Summary will update on new commits.

Review in cubic

The new flag prints the request and reply messages (verbose).
gui_rpc_debug just prints connection info and RPC commands

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 6 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread lib/cc_config.h Outdated
@AenBleidd AenBleidd added this to the Client/Manager 8.2.16 milestone Jun 24, 2026
@github-project-automation github-project-automation Bot moved this to In progress in Client/Manager Jun 24, 2026
@AenBleidd
AenBleidd merged commit 4060d3e into master Jun 24, 2026
272 checks passed
@AenBleidd
AenBleidd deleted the dpa_gui_rpc4 branch June 24, 2026 02:55
@github-project-automation github-project-automation Bot moved this from In progress to Merged in Client/Manager Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

2 participants