Skip to content

frontend_workers: remove monitoring object proxy - #284

Open
lukevalenta wants to merge 1 commit into
mainfrom
lvalenta/remove-frontend-monitoring-api
Open

frontend_workers: remove monitoring object proxy#284
lukevalenta wants to merge 1 commit into
mainfrom
lvalenta/remove-frontend-monitoring-api

Conversation

@lukevalenta

@lukevalenta lukevalenta commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove catch-all R2 object routes from the CT and Bootstrap MTC submission Workers. This keeps submission Workers limited to their documented APIs and prevents their bindings from serving monitoring data directly.
  • Advertise monitoring_url only when it is configured. Empty values are omitted instead of falling back to submission_url, so metadata does not claim that the submission Worker is also a monitoring endpoint. The generated configuration schemas now describe this behavior.
  • Preserve integration testing against both local and deployed environments. Loopback tests read checkpoints and tiles from persisted local R2 through Wrangler, while remote tests use the advertised monitoring URL. This avoids restoring the removed frontend proxy solely for tests.
  • Share the Wrangler local-R2 reader across CT, Bootstrap MTC, and mirror integration tests, and pin Wrangler 4.80.0 in CI for reproducible CLI behavior.
  • Assert that persisted checkpoints return 404 through both frontend Workers, covering the removed route behavior.
  • Sample checkpoint verification time after each fetch. A checkpoint can be created while it is being fetched; sampling beforehand made a valid new checkpoint appear future-dated, and reusing that timestamp caused every retry to fail.

Verification

  • cargo clippy --workspace --all-targets -- -Dwarnings -Dclippy::pedantic
  • cargo test
  • cargo fmt --all --check
  • cargo shear
  • cargo test -p integration_tests --test static_ct_api
  • cargo test -p integration_tests --test bootstrap_mtc_api
  • GitHub Actions: build, lint, unit tests, semgrep, CT, Bootstrap MTC, mirror, and witness integration jobs pass
@lukevalenta
lukevalenta force-pushed the lvalenta/remove-frontend-monitoring-api branch from 904d13f to 874f041 Compare August 31, 2026 21:02
@lukevalenta
lukevalenta marked this pull request as ready for review September 1, 2026 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant