It’s good that Logseq supports encryption, but as far as I can tell, it only does so for notes. Yet assets can also contain sensitive information.
While it would be great if there was universal support for encrypting and decrypting assets, I think a large chunk of that value can be obtained with less work by making use of the encryption feature built into the PDF file format - since PDFs are some of the files most likely to contain sensitive data, and images can be embedded into PDFs.
Since it is part of the file format, opening decrypted PDFs is also a fairly standard feature for PDF readers, but trying to open one at the moment causes Logseq to panic and ask the user to restart the program.