Skip to content

Cross-platform: hardcoded /tmp in cookie-import-browser.ts #708

@robertoecf

Description

@robertoecf

Spotted this while reviewing our vendored gstack install.

File: browse/src/cookie-import-browser.ts:390

The path /tmp is hardcoded here. Breaks on Windows. You already have TEMP_DIR in platform.ts for exactly this — just needs to be used here too:

const tmpPath = path.join(TEMP_DIR, "browse-cookies-" + browserName.toLowerCase() + "-" + crypto.randomUUID() + ".db");

Small fix, happy to send a PR.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions