-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PGP/Inline support #2
Comments
Thanks for my first report! Yes I would like a PR to add this. I didn't know |
For clarification, this is PGP message signing, not encryption. Right? |
The way it normally works is: you encrypt a message with my public
and then copy the output to your email window. Once I receive the email, |
Okay then. I saw the GPGTools emails sign the emails then encrypt the whole thing. Facebook does this similarly except they embed a One Pass Signature in the decrypted packets, which is signed with their own key. |
@w4 An example for the Facebook encrypted message:
Notice the
|
@w4 For a GPGTools message, the signature packet is embedded in the raw plaintext of the message. Here is the packet that is returned:
I still do not know if the message is encrypted with both keys to allow decryption by both parties. |
Though, Facebook encodes v3 signatures. GPGTools encodes v4 signatures. |
I suppose it wouldn't be too hard to tell if it was encrypted or signed first, although signed first sort of makes more sense now I come to think of it |
I may try to develop a GPG node native module. Though, this requires N1's |
There is a Node native extension that utilizes GPG's library, and I could extend off that if GPG's native API is not too terrible to analyze. |
Having a quick look at the source, it doesn't look like PGP/Inline is
supported. PGP/Inline is used a lot more now that Keybase is a thing.
You encrypt your message as you normally would, and then sign the
encrypted text with your private key. kbpgp handles everything to do
with unboxing the signed text when you call
unbox
.Would you like me to add support?
The text was updated successfully, but these errors were encountered: