Closed adidaslevy closed 1 year ago
@adidaslevy Thanks for the updated PR but it still has 2 features mixed in together. You are also missing a notifyDecryptFails: true
test case.
I think that I will split this up and merge the notifyDecryptFails
option first. The second one we can do at another stage.
Cleanup PR to only include encryptNull
option
Hey @wheresvic
I have updated my PR. Notice there is less content, and the content there is - much more ordered.
The encryptNull option enables/disables null value encryption. This is useful for saving DB disk space and computation time
The notifyDecryptFails option is used to inhibit decrypt exceptions, And allow user to receive empty field, rather than failing entire DB op. Notice that this option is not needed for node 18.