Skip to content

feature/include-all-file-types-in-personal-data-scan#170

Merged
chopkinsmade merged 1 commit into
mainfrom
feature/include-all-file-types-in-personal-data-scan
Mar 19, 2026
Merged

feature/include-all-file-types-in-personal-data-scan#170
chopkinsmade merged 1 commit into
mainfrom
feature/include-all-file-types-in-personal-data-scan

Conversation

@chopkinsmade
Copy link
Copy Markdown
Contributor

@chopkinsmade chopkinsmade commented Feb 20, 2026

Description

Reverse the logic of what files to include in a personal data scan, by defining an exclusions list instead of an inclusions list. This will allow all file types except those we know will have errors to be excluded from the scan.

Initially, this list will be limited to image files as presidio cannot scan those

Contributors

Let's acknowledge the people who contributed to the work.

Type of change

  • Refactoring (made code better without changing its behaviour)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How this has been tested

Please describe the tests that you ran to verify your changes.

If they are not automated tests please explain why and provide screenshots and/or instructions so they can reproduced.

Checklist

  • I have performed a self-review of my code
  • I have commented my code in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

Reviewer Checklist

  • I have reviewed the PR and ensured no secret values are present

@chopkinsmade chopkinsmade force-pushed the feature/include-all-file-types-in-personal-data-scan branch 8 times, most recently from b43ebfd to 109f028 Compare February 23, 2026 12:58
@chopkinsmade chopkinsmade force-pushed the feature/include-all-file-types-in-personal-data-scan branch 2 times, most recently from 868333f to d1bbcdd Compare March 2, 2026 13:17
@chopkinsmade chopkinsmade marked this pull request as ready for review March 19, 2026 09:41
@chopkinsmade chopkinsmade requested a review from a team as a code owner March 19, 2026 09:41
@chopkinsmade chopkinsmade force-pushed the feature/include-all-file-types-in-personal-data-scan branch from d1bbcdd to 60a5a85 Compare March 19, 2026 09:44
@chopkinsmade chopkinsmade merged commit c096df5 into main Mar 19, 2026
14 checks passed
@chopkinsmade chopkinsmade deleted the feature/include-all-file-types-in-personal-data-scan branch March 27, 2026 15:33
chopkinsmade added a commit that referenced this pull request Apr 28, 2026
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