-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
enhancementNew feature or requestNew feature or request
Description
Context
GET /public/feed/stream was removed in PR for issue #204 (security hardening) because:
- The path prefix
/publicis semantically wrong for an authenticated endpoint - The stream requires auth (added in Security hardening: verified findings from audit #202 #204) but lives under a route group named
/public - No clients currently use it (landing page SSE was removed in the same PR)
What to do
Create a proper GET /feed/stream SSE endpoint under a new apps/rest-api/src/routes/feed.ts route group alongside the broader cleanup of the public feed route structure:
GET /feed/stream— authenticated SSE stream (requires Bearer token)- Consider moving
GET /public/feedandGET /public/feed/searchtoGET /feedandGET /feed/searchwith auth optional (anonymous → filtered, authenticated → can passincludeSuspicious) - Update generated API client (
libs/api-client/), MCP server tools, and OpenAPI spec accordingly
References
- Removed in: Security hardening: verified findings from audit #202 #204
- Landing page human auth (cookie sessions) is a prerequisite for unauthenticated SSE consumers: #TODO
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request