Skip to content

Release 1.34.11: URL log redaction, image cache, connection tests - #114

Merged
dougrathbone merged 5 commits into
masterfrom
dougrathbone/redact-logs-cache-tests-9cfb
Sep 12, 2026
Merged

dougrathbone merged 5 commits into
masterfrom
dougrathbone/redact-logs-cache-tests-9cfb

Conversation

@dougrathbone

Copy link
Copy Markdown
Owner

Summary

Items 5-7 from the post-1.34.10 shortlist: stop logging MQTT and C-Gate download secrets, reuse BuildKit cache on the five-arch image jobs, and cover drain-timeout plus a backpressured command queue.

Changes

  • Redact URL userinfo and token-like query values before MQTT and installer logs
  • GitHub Actions BuildKit cache (per-arch scope) on CI and release image builds
  • Tests for sendWithBackpressure drain timeout/close and adaptive interval when writable connections drop
  • Changelog insert for 1.34.11 (previous headings kept)

Test plan

  • npm test passes locally with no failures
  • New code has unit test coverage (aim for ≥ 60% on changed files)
  • Existing tests were not broken or removed without justification

Checklist

  • Version bumped in package.json and homeassistant-addon/config.yaml (if releasing)
  • CHANGELOG.md updated (if releasing)
  • No sensitive data or credentials included
Open in Web Open in Cursor 

cursoragent and others added 4 commits September 12, 2026 13:40
Broker userinfo and signed query tokens were written into ordinary info and
error lines. Strip them before logging; curl and mqtt.js still get the raw
URL.

Co-authored-by: Doug Rathbone <dougrathbone@users.noreply.github.com>
Five-arch CI and release builds reconstructed every layer from scratch.
Scope the GHA cache per architecture so QEMU jobs can reuse npm and apk
layers across runs.

Co-authored-by: Doug Rathbone <dougrathbone@users.noreply.github.com>
sendWithBackpressure only had the happy drain path, and the adaptive interval
never asserted writableConnections below the healthy pool size.

Co-authored-by: Doug Rathbone <dougrathbone@users.noreply.github.com>
Co-authored-by: Doug Rathbone <dougrathbone@users.noreply.github.com>
@dougrathbone
dougrathbone marked this pull request as ready for review September 12, 2026 13:40
actionlint runs shellcheck over composite run blocks, and the new platform
resolution step tripped SC2129 with four separate appends to GITHUB_OUTPUT.

Co-authored-by: Doug Rathbone <dougrathbone@users.noreply.github.com>
@dougrathbone
dougrathbone merged commit e68ded3 into master Sep 12, 2026
25 checks passed
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.

2 participants