GitHub Security Advisory: GHSA-q926-c743-49qj
Package: github.com/centrifugal/centrifugo (GO)
Severity: LOW (CVSS 0.0)
CWE: CWE-285: Improper Authorization
Vulnerable versions: <= 2.4.0
Patched in: unpatched

Summary: Centrifugo's InsecureSkipTokenSignatureVerify flag silently disables JWT verification with no warning

Details:
### Summary
Centrifugo supports a configuration flag `insecure_skip_token_signature_verify` that completely disables JWT signature verification. When enabled, Centrifugo accepts any JWT token regardless of signature validity — including tokens signed with wrong keys, random signatures, or no signature at all. Critically, no warning is logged at startup or runtime when this flag is active, making it invisible to operators and security auditors.

Note: This vulnerability requires the operator to have explicitly set insecure_skip_token_signature_verify=true. The core issue is the absence of any warning when this flag is active, making accidental production exposure undetectable.

### Details
The flag is defined in `internal/configtypes/types.go`:
```
InsecureSkipTokenSignatureVerify bool `mapstructure:"insecure_skip_token_signature_verify"`
```

It is passed directly to token verification in `internal/client/handler.go`:
```
token, err := h.tokenVerifier.VerifyConnectToken(e.Token, 
    cfg.Client.InsecureSkipTokenSignatureVerify)
```

In `token_verifier_jwt.go`, when `skipVerify=true` the entire signature block is bypassed:
```go
if !skipVerify {
    // This block never executes
    err = verifier.verifySignature(token)
}
```
The flag is configurable via multiple vectors making accidental exposure likely:
- Config file: `insecure_skip_token_signature_verify: true`
- Environment variable: `CENTRIFUGO_INSECURE_SKIP_TOKEN_SIGNATURE_VERIFY=true`
- YAML, TOML config formats

Despite `hmac_secret_key` being configured, startup logs show `"enabled JWT verifiers"` — falsely implying verification is active.

### PoC
Config with legitimate HMAC key but skip flag enabled:
```json
{
  "client": {
    "insecure_skip_token_signature_verify": true,
    "token": { "hmac_secret_key": "legitimate-production-secret-key" }
  }
}
```

Token signed with completely wrong key is fully accepted:
```
VULNERABILITY CONFIRMED!
Connected as user: {'client': '899dec73...', 'version': '0.0.0 OSS'}
```

No security warning emitted when insecure_skip_token_signature_verify=true:
![1](https://github.com/user-attachments/assets/606acae0-e6f7-467f-b512-b5350ec6cf38)

Token signed with wrong key accepted, authentication bypass confirmed:
![2](https://github.com/user-attachments/assets/a400c0bf-b78c-40cf-8c73-07fdabb0c672)

skipVerify flag propagated from config to all token verification calls:
![3](https://github.com/user-attachments/assets/4141eb05-0371-46e1-acb7-8a9091c45693)

### Impact
- Any unauthenticated user can connect as any arbitrary user ID
- Complete authentication bypass — attacker sets any `sub` claim value
- No indicators in logs that the server is operating insecurely
- Easily triggered accidentally via environment variable injection
  in containerized deployments (e.g. misconfigured Kubernetes secrets)
- Affects all connection types: WebSocket, HTTP-streaming, SSE, GRPC
 
### Suggested Fix
1. Emit a loud startup warning when flag is enabled:
```go
if cfg.Client.Insec