iiitl / iiitl/PerOXO

Conversation auth guard

Open
#2 4 comments 0 reactions 0 assignees View on GitHub
bug easy
Dominant language
Rust
Stars
0
Forks
6
PR merge metrics
No merged PRs in 30d

Description

he GET /conversations endpoint accepts project_id, user_id_1, and user_id_2 as query params but never verifies the caller is one of those users. Anyone who can reach the endpoint can create or look up conversations between arbitrary users. Adding the same verify_token check used by the WebSocket upgrade (extract token from a header or query param, verify the caller's user_id matches one of the two) would close this.

Contributor guide

Open the contributing guide

Research direction

Locate the GET /conversations endpoint and the WebSocket upgrade path, then compare how both use verify_token and obtain the caller identity. Add coverage showing that arbitrary user pairs are rejected while an authenticated participant can access the conversation; run the existing test suite to verify the guard.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
api, authentication, authorization, backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.