GHSA-hmgr-67hw-j2cq: Open WebUI: Deactivated Channel Members Retain Full Access to Group/DM Channels
Deactivated Channel Members Retain Full Access to Group/DM ChannelsAffected ComponentChannel membership authorization check:`backend/open_webui/models/channels.py` (lines 663-673, `is_user_channel_member`)Used at 15 locations in `backend/open_webui/routers/channels.py`Affected VersionsCurrent main branch (commit `6fdd19bf1`) and likely all versions with the group/DM channel feature.DescriptionThe `is_user_channel_member` function checks whether a `ChannelMember` row exists but does not check the `is_active` field. When a user is deactivated from a group or DM channel (removed by the channel owner, or leaves voluntarily), their membership row persists with `is_active=False` and `status='left'`. Because the authorization check ignores this field, the deactivated user retains full read and write access to the channel via direct API calls.The channel correctly disappears from the deactivated user's channel list (the listing query at `get_channels_by_user_id` properly filters on `is_active`), but all 15 message-level endpoints in the router rely on `is_user_channel_member` for authorization, which does not filter on `is_active`.Compare with `get_channel_by_id_and_user_id` (line 778) which correctly checks `ChannelMember.is_active.is_(True)`.CVSS 3.1 Breakdown| Metric | Value | Rationale |
|--------|-------|-----------|
| Attack Vector | Network (N) | Exploited remotely via API calls |
| Attack Complexity | Low (L) | No special conditions beyond knowing the channel ID (which the user had as a former member) |
| Privileges Required | Low (L) | Requires a valid user account and prior channel membership |
| User Interaction | None (N) | No victim interaction required |
| Scope | Unchanged (U) | Impact is within the same authorization boundary (the channel) |
| Confidentiality | Low (L) | Can read messages in a channel the user should no longer access |
| Integrity | Low (L) | Can post, edit, and delete messages in the channel |
| Availability | None (N) | No denial of service |Attack ScenarioUser A and User B are members of a private group channel.The channel owner removes User B (or User B leaves). User B's membership is set to `is_active=False, status='left'`.The channel disappears from User B's UI — but User B noted the channel ID while they were a member.User B calls the API directly:`GET /api/v1/channels/{channel_id}/messages` — reads all messages, including those posted after deactivation`POST /api/v1/channels/{channel_id}/messages/post` — posts new messages`POST /api/v1/channels/{channel_id}/messages/{id}/update` — edits messages`DELETE /api/v1/channels/{channel_id}/messages/{id}/delete` — deletes messagesAll requests succeed because `is_user_channel_member` returns `True`.ImpactDeactivated users can continue reading all new messages posted after their removal (confidentiality breach)Deactivated users can post, edit, and delete messages (integrity breach)The deactivation mechanism provides a false sense of security — channel owners believe removed users have lost accessPreconditionsChannels feature must be enabled (disabled by default)Attacker must have a valid user accountAttacker must have been a member of the channel at some point (and thus knows the channel ID)Recommended FixAdd `is_active` filtering to `is_user_channel_member`:This aligns it with the existing `get_channel_by_id_and_user_id` method which already applies this filter correctly.
CVSS v3.1: 5.4