Redundant online_users: Vec in MessageRouter
Open
bug
easy
- Dominant language
- Rust
- Stars
- 0
- Forks
- 6
- PR merge metrics
- No merged PRs in 30d
Description
The router maintains both users:` HashMap` and `online_users: Vec
Contributor guide
Research direction
Locate MessageRouter and its GetOnlineUsers, register, and unregister paths. Read how users and online_users are maintained, then verify whether one collection can serve both purposes without changing the users returned by GetOnlineUsers. Done means registration and removal stay consistent while the redundant tracking is removed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100