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

Allow multiple anonymisation scripts at runtime #24

Open
rkm opened this issue Jan 13, 2025 · 1 comment
Open

Allow multiple anonymisation scripts at runtime #24

rkm opened this issue Jan 13, 2025 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@rkm
Copy link
Member

rkm commented Jan 13, 2025

Currently, only a single script is supported by the -a / --anon-script option. We need to support multiple scripts without restarting the service. This could be accomplished by:

  • Allowing the -a argument also to support use of a directory
  • When a directory is passed, all scripts inside the directory are loaded and registered
    • They could be registered by filename or checksum
  • When in this mode, requests to anonymise a file must also specify the script to use, returning an appropriate error if the script is unspecified or missing
@rkm rkm added the enhancement New feature or request label Jan 13, 2025
@rkm
Copy link
Member Author

rkm commented Jan 13, 2025

@jas88 Does the above implementation look ok? Would you like to have a go at implementing it?

@jas88 jas88 self-assigned this Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants