You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
No authentication dependency is present on the router or the endpoint. An
unauthenticated caller can push arbitrary event data into the SLM event
stream.
Fix
Add auth dependency to the router:
```python
from services.auth import get_current_user
router = APIRouter(
prefix="/events",
tags=["events"],
dependencies=[Depends(get_current_user)],
)
```
If `/events/sync` is intended as an internal service-to-service endpoint,
it should use service token auth from `autobot_shared` instead of being
fully open.
Discovery
Found during auth coverage audit triggered by PR #3438 (#3423 fix).
Investigated in PRs #3459 and #3472. The endpoint is intentionally unauthenticated — agents identify by node_id validated in the endpoint body. PR #3472 reverts the broken get_current_user approach and documents the security model.
Activity
fix(security): add auth to /events/sync endpoint (#3452)
fix(security): add auth to /events/sync endpoint (#3452) (#3459)
fix(security): revert get_current_user on /events/sync — agents send …
fix(security): revert get_current_user on /events/sync — agents send …
fix(security): revert get_current_user on /events/sync — agents have …
mrveiss commentedon Apr 4, 2026
Investigated in PRs #3459 and #3472. The endpoint is intentionally unauthenticated — agents identify by node_id validated in the endpoint body. PR #3472 reverts the broken get_current_user approach and documents the security model.
chore(deps): bump the npm_and_yarn group across 4 directories with 2 …