Skip to content

Exception rename - #8836

Merged
nbolton merged 10 commits into
masterfrom
exceptionRename
Aug 11, 2025
Merged

Exception rename#8836
nbolton merged 10 commits into
masterfrom
exceptionRename

Conversation

@sithlord48

@sithlord48 sithlord48 commented Aug 7, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

fixes: #8831

Uses Exception as a post fix for all previous X Items

fixes: #8830
The message is now..

FATAL: a runtime error occurred: InvalidProtocolException; Check the options section of the server configuration file has a valid protocol defined. 
Protocol must be barrier (default) or synergy.

Ex:
section: options
    protocol = barrier
end

@sithlord48
sithlord48 requested a review from nbolton August 7, 2025 21:36
@sithlord48 sithlord48 added the 🩹 fix This PR fixes a reported bug label Aug 7, 2025
nbolton
nbolton previously approved these changes Aug 11, 2025
View reviewed changes
Comment thread src/lib/arch/win32/ArchDaemonWindows.cpp Outdated
Comment thread src/lib/arch/win32/ArchDaemonWindows.cpp Outdated
Comment thread src/lib/deskflow/DeskflowException.h
@github-actions

github-actions Bot commented Aug 11, 2025 •
edited
Loading

Copy link
Copy Markdown

CI Summary

Test result

❌🔬 Tests failed: 1

OS Unit tests Legacy tests
archlinux-x86_84 ✅ Pass ✅ Pass
debian-13-arm64 ✅ Pass ✅ Pass
debian-13-x86_64 ✅ Pass ✅ Pass
fedora-41-arm64 ✅ Pass ✅ Pass
fedora-41-x86_64 ✅ Pass ✅ Pass
fedora-42-arm64 ✅ Pass ✅ Pass
fedora-42-x86_64 ✅ Pass ✅ Pass
macos-13-x64 ✅ Pass ✅ Pass
macos-14-arm64 ✅ Pass ✅ Pass
opensuse-arm64 ✅ Pass ✅ Pass
opensuse-x86_84 ✅ Pass ✅ Pass
ubuntu-25.04-arm64 ✅ Pass ✅ Pass
ubuntu-25.04-x86_64 ✅ Pass ✅ Pass
windows-2022-arm64 ❌ Fail ✅ Pass
windows-2022-x64 ✅ Pass ✅ Pass

Full summary (scroll down)

@nbolton
nbolton enabled auto-merge (rebase) August 11, 2025 16:08
@nbolton
nbolton merged commit 6840e1e into master Aug 11, 2025
29 of 30 checks passed
@nbolton
nbolton deleted the exceptionRename branch August 11, 2025 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nbolton nbolton nbolton approved these changes

Assignees

No one assigned

Labels

🩹 fix This PR fixes a reported bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Drop X prefix on exception classes Improve confusing XInvalidProtocol error message

2 participants

@sithlord48 @nbolton