Repository navigation
Conversation
bb-block staging moved from test.blockstaging.build to block.builderlabstaging.xyz, so `bb apps` must accept compose-ctrl.block.builderlabstaging.xyz. The old host still answers during the transition and stays trusted. Co-authored-by: Jitter <2b2e6415e748c35180847a37aadae06a11e30dddc76b784e7fd9354c4eb42e7a@buzz.block.builderlab.xyz> Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
bb-block staging moved from test.blockstaging.build to block.builderlabstaging.xyz. Move the staging OTLP endpoint to otel.block.builderlabstaging.xyz at all four sites that pin it: the vite.config.ts default, ALLOWED_OTEL_LOGS_HOSTS, the CSP connect-src and the pinned tests. Co-authored-by: Jitter <2b2e6415e748c35180847a37aadae06a11e30dddc76b784e7fd9354c4eb42e7a@buzz.block.builderlab.xyz> Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
morgmart
left a comment
There was a problem hiding this comment.
🤖 Automated code review
Engineering found no concrete changed-line defects in the exact PR comparison. The new staging domains are applied consistently across the CLI allowlist and tests, the telemetry build endpoint, native allowlist and tests, CSP, and documentation. Supplied GitHub evidence is structurally valid; Frontend checks and unit tests were still in progress while the other reported checks were successful or skipped, so required checks still govern merge readiness.
Deterministic publication result: 0 blocking and 0 non-blocking inline finding(s) publishable; 0 duplicate(s) suppressed; 0 blocking screenshot-evidence requirement(s) in this review body.
Pending checks: 2 check(s) are not complete.
This approval reflects the completed code review only; merge readiness remains governed by the repository's required checks.
Summary
bb-block staging moved from
test.blockstaging.buildtoblock.builderlabstaging.xyz. The cutover finished on 2026-10-27 with BPCI #2028. This PR has two commits:compose-ctrl.block.builderlabstaging.xyztoTRUSTED_CONTROL_PLANE_HOSTS. The oldcompose-ctrl.test.blockstaging.buildhost still answers (401without credentials), so it stays trusted until the old domain is retired.otel.block.builderlabstaging.xyz. Per the note invite.config.ts, the host changes at all four sites together: the default endpoint,ALLOWED_OTEL_LOGS_HOSTS, the CSPconnect-srcand the pinned tests. Both otel hosts are routed by the staging ingress, andPOST /v1/logson the new host returns the same415as the old one.Related issue
N/A. Same change for builderlab-cli: block/builderlab-cli#36.
Testing
bb appsallowlist test (control_plane_allowlist_is_exact_and_https_only) passes. It covers the new host with and without:443and rejects itshttp, attacker-suffix and:444variants.block-telemetry-enforced(46 and 38 tests).just checkpasses.sqlx-macros: "mis-aligned LINKEDIT string pool"). So I ran the Tauri tests with Rust 1.98.1 and pushed with thetauri-checkandclippyhooks excluded. Clippy on 1.98.1 flags only pre-existing new lints incrates/berd-call, none in the files changed here. CI with the pinned toolchain is the authoritative check.🤖 Generated with Claude Code