[PR #664] [MERGED] Change log level on several status messages (Fixes #652, Fixes #516) #1673

New issue

Closed

opened 2026-05-05 07:59:08 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 07:59:08 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/debauchee/barrier/pull/664
Author: @simons-public
Created: 5/12/2020
Status: ✅ Merged
Merged: 5/12/2020
Merged by: @shymega

Base: master ← Head: fix-gui-status

📝 Commits (3)

a9421a9 Change log level on several status messages (Fixes #652, Fixes #516)
5636e9c A few more log cleanups
639415e Merge pull request #2 from plessbd/patch-3

📊 Changes

4 files changed (+7 additions, -8 deletions)

View changed files

📝 src/gui/src/MainWindow.cpp (+4 -5)
📝 src/lib/barrier/ClientApp.cpp (+1 -1)
📝 src/lib/barrier/ServerApp.cpp (+1 -1)
📝 src/lib/barrier/win32/DaemonApp.cpp (+1 -1)

📄 Description

This changes the log level for several statuses (started server, connected to server, server status:) from CLOG_NOTE or CLOG_INFO to CLOG_PRINT so they will be printed to stdout regardless of the log level. This allows the GUI to accurately report the status of the barriers or barrierc processes in MainWindow.cpp#L379-L399

Until the IPC connection state messages are implemented this should fix any issues with the GUI not reporting the correct status.

Example output with this change running barriers (and ignoring TIS/TSM #653)

./build/bin/barriers -f --no-tray --debug WARNING --name <hostname> --enable-drag-drop --enable-crypto -c  <config>  --address :24800 2>&1 | egrep -v TSM
started server (IPv4/IPv6), waiting for clients

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/debauchee/barrier/pull/664 **Author:** [@simons-public](https://github.com/simons-public) **Created:** 5/12/2020 **Status:** ✅ Merged **Merged:** 5/12/2020 **Merged by:** [@shymega](https://github.com/shymega) **Base:** `master` ← **Head:** `fix-gui-status` --- ### 📝 Commits (3) - [`a9421a9`](https://github.com/debauchee/barrier/commit/a9421a9cd3539a6197d060eb1ad8d79b29d946a6) Change log level on several status messages (Fixes #652, Fixes #516) - [`5636e9c`](https://github.com/debauchee/barrier/commit/5636e9cb404c63cd954aff778520c999d2da194e) A few more log cleanups - [`639415e`](https://github.com/debauchee/barrier/commit/639415ea442f727bc1b79e3a2785da35484b9f7b) Merge pull request #2 from plessbd/patch-3 ### 📊 Changes **4 files changed** (+7 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `src/gui/src/MainWindow.cpp` (+4 -5) 📝 `src/lib/barrier/ClientApp.cpp` (+1 -1) 📝 `src/lib/barrier/ServerApp.cpp` (+1 -1) 📝 `src/lib/barrier/win32/DaemonApp.cpp` (+1 -1) </details> ### 📄 Description This changes the log level for several statuses (started server, connected to server, server status:) from CLOG_NOTE or CLOG_INFO to CLOG_PRINT so they will be printed to stdout regardless of the log level. This allows the GUI to accurately report the status of the `barriers` or `barrierc` processes in [MainWindow.cpp#L379-L399](https://github.com/debauchee/barrier/blob/master/src/gui/src/MainWindow.cpp#L379-L399) Until the IPC connection state messages are implemented this should fix any issues with the GUI not reporting the correct status. Example output with this change running `barriers` (and ignoring TIS/TSM #653) ``` ./build/bin/barriers -f --no-tray --debug WARNING --name <hostname> --enable-drag-drop --enable-crypto -c <config> --address :24800 2>&1 | egrep -v TSM started server (IPv4/IPv6), waiting for clients ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>