Skip to content

fix(scan): encrypted/corrupt files → scan_failures (#828 open-core)#1145

Closed
FabioLeitao wants to merge 2 commits into
mainfrom
feat/828-scan-failures-encrypted
Closed

fix(scan): encrypted/corrupt files → scan_failures (#828 open-core)#1145
FabioLeitao wants to merge 2 commits into
mainfrom
feat/828-scan-failures-encrypted

Conversation

@FabioLeitao

Copy link
Copy Markdown
Collaborator

Summary

Open-core slice of #828não-verificável ≠ limpo (twin of merged #1140 SQL sampling_error).

  • Zip/7z: iter_archive_memberssave_failure (encrypted_no_password, wrong_password, archive_read_error).
  • PDF: encrypted without password / wrong decrypt → same taxonomy.
  • failure_hint for new reasons.

Follow-up (same issue, later PRs/commits)

  • Real encrypted fixtures; broader corrupt doc paths; PLAN_PASSWORD_PROTECTED_SCANNING.md; Maestro; Pro decrypt.

Test plan

  • tests/test_archives_scan_failures.py
  • ./scripts/check-all.sh green

Partial toward #828 — does not close full AC yet.

Made with Cursor

Zip/7z member read failures call on_member_read_failure with encrypted_no_password,
wrong_password, or archive_read_error. PDF without password uses the same taxonomy.
Open-core slice; Pro decrypt and plan doc follow in later commits.
@FabioLeitao

Copy link
Copy Markdown
Collaborator Author

OPSEC: branch history had accidental private files in 706277a; superseded by clean branch feat/828-scan-failures-clean (no force-push allowed on old ref).

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.

1 participant