docs(backend): refresh SSE endpoint guidance - #995
Conversation
|
CI audit: the docs-specific validation and both CodeQL jobs pass. The remaining matrix failures occur before or outside this README change: backend/frontend/Docker dependency installation fails on the existing Vitest 3.2.7 vs @vitest/coverage-v8 2.1.9 peer conflict, and contract jobs fail in existing Rust dependencies/tests. This PR changes only �ackend/SSE_README.md. |
a86e678 to
428c99f
Compare
428c99f to
d21ec71
Compare
|
Hi maintainers! 👋 This PR branch has been cleanly rebased onto the latest main branch with 0 merge conflicts. Ready for review and merge! |
ogazboiz
left a comment
There was a problem hiding this comment.
most of this checks out against the code (v1 subscribe with JWT, stats endpoint, redis pub/sub, per-IP capacity). two fixes:
- the claim that the old unversioned GET /events/subscribe returns HTTP 410 Gone is not true, there is no 410 anywhere in the backend; unversioned paths fall through to the deprecated handlers or 404. document the actual behavior.
- #1001 also creates backend/SSE_README.md, whichever merges second needs a rebase, so coordinate.
if you want to keep contributing, join us on Telegram: https://t.me/+DOylgFv1jyJlNzM0
Summary
Closes #848
Validation
Docs only; docs/SSE_ARCHITECTURE.md was not changed.