Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to pseudonymize the common name #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

phihos
Copy link

@phihos phihos commented Jun 4, 2022

I finally got around to implement the feature. The changes were bigger than expected. Sorry about that in advance.

The command module now has end-to-end tests. For that to work in the same process I had to make some changes to the command module itself. But they should be functionally equivalent. Since the changes only work on newer golang versions I had to update the minimum version.

The pseudonymization feature itself is implemented as decorator for parser.ParseFile. If applied it intercepts the Status struct and pseudonymizes each CommonName field in the ClientList.
As promised everything is fully tested. I did not touch any preexisting test so you can be confident that the previous behavior has not changed.

Closes #19

@phihos phihos force-pushed the 19 branch 8 times, most recently from fc62e8b to b8fe910 Compare June 5, 2022 18:16
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.

Pseudonymization of common_name
1 participant