Skip to content

fix(config): validate protected cookie stores - #386

Open
iMelki wants to merge 2 commits into
nesszer:mainfrom
iMelki:agent/codexbar/secure-file-validator-20260825
Open

fix(config): validate protected cookie stores#386
iMelki wants to merge 2 commits into
nesszer:mainfrom
iMelki:agent/codexbar/secure-file-validator-20260825

Conversation

@iMelki

@iMelki iMelki commented Aug 25, 2026

Copy link
Copy Markdown

Summary

  • Validate configuration through the same secure-file reader used by the application.
  • Accept Windows DPAPI-protected manual-cookie stores without converting them to plaintext.
  • Add focused coverage for a valid protected store and malformed-wrapper error redaction.

Validation

  • git diff --check
  • cargo fmt --all --check
  • gitleaks protect --staged --redact
  • Focused Rust test compilation is currently blocked locally before tests start: Cargo returns Windows os error 2 while copying the num-traits build script. The failed build cache and logs were preserved; no security product or source data was changed.
  • The upstream PR check workflow is awaiting maintainer approval because this is a first-time fork contribution.

Privacy

This change does not include account cookies, credential values, personal paths, or local configuration data.

@Finesssee
Finesssee marked this pull request as ready for review August 25, 2026 17:56
@Finesssee

Copy link
Copy Markdown
Collaborator

Thanks for the PR, I will review it ASAP.

@iMelki
iMelki marked this pull request as draft August 25, 2026 18:05
@Finesssee
Finesssee marked this pull request as ready for review August 25, 2026 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants