Skip to content

Conversation

patheard
Copy link
Member

@patheard patheard commented Aug 26, 2025

Summary | Résumé

Update to the latest version of the @cdssnc/sanitize-pii module so that error code timestamps are no longer partially redacted and incorrectly identified as phone numbers.

Related

Test instructions | Instructions pour tester la modification

  1. After deploy, create a support request that includes the Forms error code format.
  2. Expect that the timestamp portion of the error code is not redacted.

Unresolved questions / Out of scope | Questions non résolues ou hors sujet

None.

Pull Request Checklist

Please complete the following items in the checklist before you request a review:

  • Have you completely tested the functionality of change introduced in this PR? Is the PR solving the problem it's meant to solve within the scope of the related issue?
  • The PR does not introduce any new issues such as failed tests, console warnings or new bugs.
  • If this PR adds a package have you ensured its licensed correctly and does not add additional security issues?
  • Is the code clean, readable and maintainable? Is it easy to understand and comprehend.
  • Does your code have adequate comprehensible comments? Do new functions have docstrings?
  • Have you modified the change log and updated any relevant documentation?
  • Is there adequate test coverage? Both unit tests and end-to-end tests where applicable?
  • If your PR is touching any UI is it accessible? Have you tested it with a screen reader? Have you tested it with automated testing tools such as axe?

Update to the latest version of the `@cdssnc/sanitize-pii` module so that
error code timestamps are no longer partially redacted and incorrectly
identifed as phone numbers.
@patheard patheard self-assigned this Aug 26, 2025
Copy link
Contributor

@patheard patheard merged commit f452a1d into main Aug 26, 2025
15 of 17 checks passed
@patheard patheard deleted the fix/timestamp-sanitize branch August 26, 2025 14:48
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