Closed TheMrMilchmann closed 1 year ago
(Pinging @Spasi to have a look at this because I'm not sure if GitHub notifies people about draft PRs. No hurry though. 😅)
Hey @TheMrMilchmann,
I've added a commit that addresses the above questions. Will split/squash/cleanup the commits as necessary when it's time to merge this.
Thanks, I just looked at your changes and I believe you have already taken care of all remaining issues. In my opinion, this is good to go. However, I'm not sure where to mention this in the changelog. Could you also take care of this? Otherwise, let me know if anything else is missing.
Thanks @TheMrMilchmann!
Did one more change before the merge, added IgnoreMissing..
to CryptProtectMemory
and CryptUnprotectMemory
because technically we support Windows XP and these two are available on Vista+.
Bindings for dpapi.h and crypt32.dll providing support for:
CryptProtectData
andCryptUnprotectData
, andCryptProtectMemory
andCryptUnprotectMemory
.This is not fully finished yet and I'll add some questions inline.