GHSA-m93w-4fxv-r35v

Suggest an improvement
Source
https://github.com/advisories/GHSA-m93w-4fxv-r35v
Import Source
https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2024/06/GHSA-m93w-4fxv-r35v/GHSA-m93w-4fxv-r35v.json
JSON Data
https://api.osv.dev/v1/vulns/GHSA-m93w-4fxv-r35v
Aliases
Published
2024-06-18T20:29:33Z
Modified
2024-07-05T21:42:20Z
Severity
  • 5.4 (Medium) CVSS_V3 - CVSS:3.1/AV:N/AC:L/PR:N/UI:R/S:U/C:L/I:L/A:N CVSS Calculator
Summary
PocketBase performs password auth and OAuth2 unverified email linking
Details

In order to be exploited you must have both OAuth2 and Password auth methods enabled.

A possible attack scenario could be: - a malicious actor register with the targeted user's email (it is unverified) - at some later point in time the targeted user stumble on your app and decides to sign-up with OAuth2 (this step could be also initiated by the attacker by sending an invite email to the targeted user) - on successful OAuth2 auth we search for an existing PocketBase user matching with the OAuth2 user's email and associate them - because we haven't changed the password of the existing PocketBase user during the linking, the malicious actor has access to the targeted user account and will be able to login with the initially created email/password

To prevent this for happening we now reset the password for this specific case if the previously created user wasn't verified (an exception to this is if the linking is explicit/manual, aka. when you send Authorization:TOKEN with the OAuth2 auth call).

Additionally to warn existing users we now send an email alert in case the user has logged in with password but has at least one OAuth2 account linked. It looks something like:

Hello, Just to let you know that someone has logged in to your Acme account using a password while you already have OAuth2 GitLab auth linked. If you have recently signed in with a password, you may disregard this email. If you don't recognize the above action, you should immediately change your Acme account password. Thanks, Acme team

The flow will be further improved with the ongoing refactoring and we will start sending emails for "unrecognized device" logins (OTP and MFA is already implemented and will be available with the next v0.23.0 release in the near future).

Database specific
{
    "github_reviewed_at": "2024-06-18T20:29:33Z",
    "cwe_ids": [
        "CWE-287"
    ],
    "nvd_published_at": "2024-06-18T17:15:52Z",
    "severity": "MODERATE",
    "github_reviewed": true
}
References

Affected packages

Go / github.com/pocketbase/pocketbase

Package

Name
github.com/pocketbase/pocketbase
View open source insights on deps.dev
Purl
pkg:golang/github.com/pocketbase/pocketbase

Affected ranges

Type
SEMVER
Events
Introduced
0Unknown introduced version / All previous versions are affected
Fixed
0.22.14