Recipe loading failure should result as an exception

[travnick]

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.

[vsoch]

You could also just check to see that the recipe is not None on the class.

[travnick]

Oh, ok, thank you! Considering this as a workaround.