Skip to content

feat: add protocol harness that sweeps the wire format versions - #14

Open
nbolton wants to merge 2 commits into
mainfrom
feat/protocol-harness
Open

nbolton wants to merge 2 commits into
mainfrom
feat/protocol-harness

Conversation

@nbolton

@nbolton nbolton commented Sep 18, 2026

Copy link
Copy Markdown
Member

Generated by Claude

Companion to deskflow/deskflow#10069, the key repeat protocol fix. No Jira issue.

Test steps

  1. With a Deskflow server running from the GUI and a screen named harness in its layout, run python3 protocol_harness.py client --name harness --versions 1.8,1.6 --wait 20. When it prompts, push the pointer onto the harness screen, hold a key for two seconds, then move back.

    Expect: ok DKRP lines and PASS for both versions on the fix branch; on master the 1.6 run reports six leftover bytes and fails.

  2. Run python3 protocol_harness.py server, then start a Deskflow client from the GUI pointed at this machine.

    Expect: 1.8, 1.7 and 1.6 each fully "parsed and answered", 1.5 refused, PASS on all four.

Fake peer that acts as a client or a server at each protocol version and checks every message against the formats Synergy 1.x, Barrier and Input Leap use, so a mismatch both halves of Deskflow agree on still fails.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant