Skip to content

Vikunja: WebSocket authentication ignores server-side session state, so revoked sessions keep receiving live pushes

Moderate severity GitHub Reviewed Published Oct 3, 2026 in go-vikunja/vikunja • Updated Oct 9, 2026

Package

gomod code.vikunja.io/api (Go)

Affected versions

>= 2.3.0, <= 2.6.0

Patched versions

None

Description

Summary

Vikunja v2.6.0 introduced server-side sessions with revocation semantics: DELETE /api/v2/user/sessions/{id} documents "Revokes a specific session by its UUID", and enabling TOTP calls DeleteAllUserSessions to invalidate all sessions. The WebSocket endpoint accepts any cryptographically valid user JWT without ever resolving its sid (session id) claim against the sessions table. A connection authenticated before revocation stays authenticated and continues to receive live pushes indefinitely, and even brand-new WebSocket connections are accepted with a token whose session was deleted. The revocation feature therefore does not cover the WebSocket boundary at all.

Details

User access JWTs carry a session reference: NewUserJWTAuthtoken (pkg/modules/auth/auth.go:188) sets claims["sid"] = sessionID, and the API login-session documentation describes the claim as tying the JWT to a server-side session. On the HTTP side nothing consumes the claim per request either (REST access tokens simply expire after their short TTL, a design the maintainers documented when fixing GHSA-96q5, and refresh tokens are correctly invalidated when the session row is deleted).

The WebSocket endpoint GET /api/v2/ws authenticates through the first client message. Connection.handleAuth (pkg/websocket/connection.go:168) validates the token exclusively with auth.GetUserIDFromToken (pkg/modules/auth/auth.go:292):

func GetUserIDFromToken(tokenString string) (int64, error) {
	token, err := jwt.Parse(tokenString, func(_ *jwt.Token) (any, error) {
		return []byte(config.ServiceSecret.GetString()), nil
	})
	...
	typ, ok := claims["type"].(float64)
	if !ok || int(typ) != AuthTypeUser {
		return 0, jwt.ErrTokenInvalidClaims
	}
	...
}

The function verifies signature and token type only. No session lookup occurs anywhere in the WebSocket path, and no re-validation happens after the initial auth. Consequently:

  1. A connection authenticated before DELETE /api/v2/user/sessions/{id} (or before TOTP enable, which runs models.DeleteAllUserSessions, pkg/routes/api/v2/user_totp.go:208) remains registered in the hub and keeps receiving every subscribed event, such as notification.created with the full notification object including comment, task and project content.
  2. A new connection opened after the deletion, using the same pre-revocation token, authenticates successfully.
  3. Session deletion is observable only in the REST surface. The user's session list empties, the refresh token dies on next use, but the WebSocket channel never notices.

For REST the exposure is bounded by the 10-minute access-token TTL. For WebSocket it is not: a connection lives until the client closes it, so a session the user believes revoked retains a live, server-pushed data channel with no time bound.

PoC

vikunja-ws-auth-ignores-session-revocation-PoC.zip

The attached archive contains deployment/deploy.sh (local Vikunja v2.6.0 on loopback with SQLite) and poc/poc.sh running poc/poc.py:

  1. Victim registers and logs in. Second user registers for later interaction.
  2. Control: the victim's WebSocket authenticates and subscribes to notification.created. Negative control: a garbage token is rejected with invalid_token.
  3. Every victim session is deleted via DELETE /api/v2/user/sessions/{id} (the list afterwards reports 0 remaining).
  4. Exploit A: a brand-new WebSocket connection authenticates successfully with the deleted-session token.
  5. Exploit B: the second user comments on a task mentioning the victim. The victim's pre-revocation WebSocket connection receives the notification.created push with the full notification payload.

Result: 10 checks passed on two consecutive runs, each from a wiped database.

Impact

Session revocation, the mechanism a user relies on when a device is lost or a token is suspected stolen, silently excludes the WebSocket channel. A stolen or leaked access token can be converted into a push channel that survives every revocation action available to the user, including TOTP enrollment whose documented purpose is invalidating all sessions, and keeps delivering the victim's notifications (comment mentions, task and project metadata) until the attacker chooses to disconnect. The fix is to resolve the sid claim against the sessions table during handleAuth and to re-validate or drop hub connections when their session is deleted.

References

@kolaente kolaente published to go-vikunja/vikunja Oct 3, 2026
Published to the GitHub Advisory Database Oct 9, 2026
Reviewed Oct 9, 2026
Last updated Oct 9, 2026

Severity

Moderate

CVSS overall score

This score calculates overall vulnerability severity from 0 to 10 and is based on the Common Vulnerability Scoring System (CVSS).
/ 10

CVSS v3 base metrics

Attack vector
Network
Attack complexity
Low
Privileges required
Low
User interaction
None
Scope
Unchanged
Confidentiality
High
Integrity
None
Availability
None

CVSS v3 base metrics

Attack vector: More severe the more the remote (logically and physically) an attacker can be in order to exploit the vulnerability.
Attack complexity: More severe for the least complex attacks.
Privileges required: More severe if no privileges are required.
User interaction: More severe when no user interaction is required.
Scope: More severe when a scope change occurs, e.g. one vulnerable component impacts resources in components beyond its security scope.
Confidentiality: More severe when loss of data confidentiality is highest, measuring the level of data access available to an unauthorized user.
Integrity: More severe when loss of data integrity is the highest, measuring the consequence of data modification possible by an unauthorized user.
Availability: More severe when the loss of impacted component availability is highest.
CVSS:3.1/AV:N/AC:L/PR:L/UI:N/S:U/C:H/I:N/A:N

EPSS score

Weaknesses

Insufficient Session Expiration

According to WASC, Insufficient Session Expiration is when a web site permits an attacker to reuse old session credentials or session IDs for authorization. Learn more on MITRE.

CVE ID

No known CVE

GHSA ID

GHSA-4hv6-xc92-j86g

Source code

Credits

Loading Checking history
See something to contribute? Suggest improvements for this vulnerability.