Skip to content

Comments

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

Draft
chopkinsmade wants to merge 1 commit intomainfrom
feature/include-all-file-types-in-personal-data-scan
Draft

feature/include-all-file-types-in-personal-data-scan#170
chopkinsmade wants to merge 1 commit intomainfrom
feature/include-all-file-types-in-personal-data-scan

Conversation

@chopkinsmade
Copy link
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 7 times, most recently from 59c97c9 to b43ebfd Compare February 23, 2026 12:48
@chopkinsmade chopkinsmade force-pushed the feature/include-all-file-types-in-personal-data-scan branch from b43ebfd to 109f028 Compare February 23, 2026 12:58
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