Skip to content

fix(logs): cover all declared redacted fields - #103

Merged
frobel0520 merged 1 commit into
mainfrom
feature/logs-redaction-coverage
Aug 23, 2026
Merged

fix(logs): cover all declared redacted fields#103
frobel0520 merged 1 commit into
mainfrom
feature/logs-redaction-coverage

Conversation

@frobel0520

@frobel0520 frobel0520 commented Aug 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • add password, accessToken and cookie to the redaction fixture contract
  • map every declared sensitive field in the simulator verifier
  • test raw-value leakage for every declared field
  • update Logs lesson and acceptance evidence

Verification

  • npm --prefix frontend test -- src/topics/logs/content.test.ts src/topics/logs/simulator.test.ts src/topics/logs/lab.test.ts src/topics/logs/integration.test.ts
  • npm --prefix frontend run lint

@frobel0520
frobel0520 merged commit a2618c7 into main Aug 23, 2026
2 checks passed
@frobel0520
frobel0520 deleted the feature/logs-redaction-coverage branch August 23, 2026 03:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@frobel0520