enbility / ship-go

93%

LAST BUILD ON BRANCH dev

branch: dev

CHANGE BRANCH

x

Reset

• dev
• auto_accept_stalls_ship
• bug/attempt_reconnect_when_device_reappears
• bug/autoAcceptAnnouncement
• bug/avahi_dead_lock
• bugfix/race-condition
• chore/redundant-nil-checks
• docs/testspec-coverage-comments
• feat/dynamicMdnsInterfaceManagement
• feat/hub-bound-port
• feature/action-update
• feature/analysis-docs
• feature/bugfixes
• feature/hub-improvements
• feature/json-parsing-improvements
• feature/maintenance
• feature/mdns-restart-improvements
• feature/register-improvements
• feature/shippairing-2
• feature/tls-improvements
• fix/double-connection-outgoing-dedup
• fix/hub-connection-bugs
• fix/hub_connections_test
• fix/json-whitespace-tolerance
• fix/mdns-service-type-case-sensitivity
• fix/pairing-issues-and-trust-dupes
• fix/pairing-reactivate-listener-on-addCu-removal
• fix/pairing-startup-listener-guard
• fix/pairing-txt-format-validation
• fix/prevent-stuck-shutdown-test-goroutine-leak
• fix/reannounce-entrygroup-leak
• fix/remove-unused-announcement-cancel-ctx
• fix/ship-double-connection
• fix/ship-handshake-conformance
• fix/ship-tests-tc-ship-amdata-002
• fix/spine-write-error-teardown
• fix/trusted-reconnect-mdns-drop
• fix_issue_49
• main
• safer-connection-attempt-running-handling
• update-to-zeroconf-v3
• upstream/reset-connection-backoff

Build # 30808179265

Build Type

push

github

Committed by web-flow

Commit Message

Tolerate structural whitespace in EEBUS JSON (#100)

JsonFromEEBUSJson unwraps the EEBUS array-of-single-key-objects encoding
via exact byte-sequence replacements. Any RFC 8259 structural whitespace
around JSON tokens defeated those matches, leaving the outer array
intact so the downstream json.Unmarshal failed.

Normalise the input with json.Compact before the replacements. Compact
is string-aware, so whitespace inside string values is preserved;
invalid JSON falls back to the raw bytes so behaviour is no worse than
before. The NUL trim moves ahead of Compact since a trailing 0x00 is
invalid JSON.

Add whitespace-tolerance tests covering all four whitespace characters
around structural tokens, whitespace inside string values, and a
pretty-printed real datagram.

Fixes #97.

Coverage Stats

11 of 11 new or added lines in 1 file covered. (100.0%)

5 existing lines in 2 files now uncovered.

5934 of 6399 relevant lines covered (92.73%)

12224.2 hits per line