Fix XGB reply-cookie liveness checks #14

Merged
piotr merged 1 commits from issue-2-xgb-reply-cookie into main 2026-10-02 10:43:54 +02:00
Collaborator

Follow-up root-cause fix for the Orange Pi reconnect loop reported in #2.

Summary

  • consume reply-bearing GetInputFocus cookies with Reply() in both post-map synchronization and session liveness checks
  • propagate liveness errors while preserving mapped-window cleanup and reconnect containment
  • add behavioral tests using the real XGB cookie API to prevent regressions to invalid Check() usage

Validation

  • go test -race ./internal/display
  • go test ./...
  • go vet ./... and gofmt -l
  • ARM64 builds of both binaries
  • git diff --check and added-lines secrets scan

Physical acceptance after merge: one successful kiosk window and no recurring open_failed loop.

Follow-up root-cause fix for the Orange Pi reconnect loop reported in #2. ## Summary - consume reply-bearing `GetInputFocus` cookies with `Reply()` in both post-map synchronization and session liveness checks - propagate liveness errors while preserving mapped-window cleanup and reconnect containment - add behavioral tests using the real XGB cookie API to prevent regressions to invalid `Check()` usage ## Validation - `go test -race ./internal/display` - `go test ./...` - `go vet ./...` and `gofmt -l` - ARM64 builds of both binaries - `git diff --check` and added-lines secrets scan Physical acceptance after merge: one successful kiosk window and no recurring `open_failed` loop.
grzesiu self-assigned this 2026-10-02 10:42:52 +02:00
grzesiu added 1 commit 2026-10-02 10:42:52 +02:00
fix: resolve reply-bearing X11 liveness cookies with Reply (#2)
Build Ground Station / build (push) Successful in 26s
ffae3560c4
piotr merged commit ffae3560c4 into main 2026-10-02 10:43:54 +02:00
piotr deleted branch issue-2-xgb-reply-cookie 2026-10-02 10:43:55 +02:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: piotr/groundstation#14