deid icon indicating copy to clipboard operation
deid copied to clipboard

Recipe loading failure should result as an exception

Open travnick opened this issue 9 months ago • 2 comments

Currently, failure in loading a recipe by DeidRecipe class, generates WARNING Problem loading deid_default.recipe, skipping. log message only.

I consider this failure as an error, since my carefully handcrafted anonymization rules are not loaded, and the file is not processed correctly. Since this is the main point of the project, then it's an ERROR, not a WARNING

As it's the error, then the exception should be raised - further processing is not possible. Using any defaults in this case may lead to sensitive data leakage. I'm aware of that project gives no guarantees. But currently, user is not able to handle the error in any way.

travnick avatar May 24 '24 14:05 travnick