GHSA-wvh7-5p38-2qfc

Suggest an improvement
Source
https://github.com/advisories/GHSA-wvh7-5p38-2qfc
Import Source
https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2020/07/GHSA-wvh7-5p38-2qfc/GHSA-wvh7-5p38-2qfc.json
JSON Data
https://api.osv.dev/v1/vulns/GHSA-wvh7-5p38-2qfc
Published
2020-07-23T18:20:10Z
Modified
2021-09-22T21:05:43Z
Summary
Storing Password in Local Storage
Details

The setPassword method (http://parseplatform.org/Parse-SDK-JS/api/2.9.1/Parse.User.html#setPassword) stores the user's password in localStorage as raw text making it vulnerable to anyone with access to your localStorage. We believe this is the only time that password is stored at all. In the documentation under Users > Signing Up, it clearly states, "We never store passwords in plaintext, nor will we ever transmit passwords back to the client in plaintext."

Example Code:

async () => {
    const user = Parse.User.current()
    if (user) {
        user.setPassword('newpass')
        await user.save()
    }
}

After running the above code, the new password will be stored in localStorage as a property named "password".

Proposed Solution: Before saving anything to localStorage, Parse should strip out any properties named "password" that are attempting to be stored with a Parse.User type object.

Configuration: Parse SDK: 2.9.1 Parse Server: 3.9.0

Database specific
{
    "cwe_ids":  [
        "CWE-256"
    ],
    "github_reviewed":  true,
    "github_reviewed_at":  "2020-07-23T18:18:24Z",
    "nvd_published_at":  null,
    "severity":  "MODERATE"
}
References

Affected packages

npm / parse

Package

Affected ranges

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

Database specific

source
"https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2020/07/GHSA-wvh7-5p38-2qfc/GHSA-wvh7-5p38-2qfc.json"