Password confirmation stored in plain text via registration form in statamic/cms
Impact
Users registering via the user:register_form tag will have their password confirmation stored in plain text in their user file.
Source excerpt · OSVImpact
This only affects sites matching all of the following conditions:
- Running Statamic versions between 5.3.0 and 5.6.1.
Affected packages
Start with the recorded fix for your release branch. Check your installed version and test compatibility before upgrading. Fixing this advisory does not rule out other vulnerabilities in the release.
| Affected range | Recorded fix |
|---|---|
| From 5.3.0 before 5.6.2Ecosystem versions | 5.6.2 |
Version evidence: OSV · GHSA-qvpj-w7xj-r6w9
View original version evidence (5 explicit versions)
Original source description
Users registering via the user:register_form tag will have their password confirmation stored in plain text in their user file.
Impact
This only affects sites matching all of the following conditions:
- Running Statamic versions between 5.3.0 and 5.6.1. (This version range represents only one calendar week)
- Using the
user:register_formtag. - Using file-based user accounts. (Does not affect users stored in a database.)
- Has users that have registered during that time period. (Existing users are not affected.)
The password is only visible to users that have access to read user yaml files, typically developers of the application itself.
Patches
The issue has been patched in 5.6.2, however any users registered during that time period and using the affected version range will still have the the password_confirmation value in their yaml files.
We recommend that affected users have their password reset. The following query can be entered into php artisan tinker and will output a list of affected emails:
Statamic\Facades\User::query()->whereNotNull('password_confirmation')->get()->map->email
The following can be entered into tinker and will clear both password_confirmation as well as their existing password. They will be required to reset their password before their next login attempt.
Statamic\Facades\User::query()
->whereNotNull('password_confirmation')->get()
->each(fn ($user) => $user->remove('password_confirmation')->passwordHash(null)->save());
References
If you are committing user files to a public git repo, you may consider clearing the sensitive data from the git history. You can use the following links for details.
References
- Security advisory github.com/statamic/cms/security/advisories/GHSA-qvpj-w7xj-r6w9
- Security advisory nvd.nist.gov/vuln/detail/CVE-2024-36119
- Patch github.com/statamic/cms/commit/0b804306c96c99b81755d5bd02df87ddf392853e
- Source reference dev.to/balogh08/cleaning-your-git-history-safely-removing-sensitive-data-10i5
- Source reference docs.github.com/en/authentication/keeping-your-account-and-data-secure/removing-sensitive-data-from-a-repository
- Source reference github.com/statamic/cms