decentralized-identity / didcomm-messaging

https://identity.foundation/didcomm-messaging/spec/
Apache License 2.0
161 stars 56 forks source link

fix 236 #386

Closed dhh1128 closed 2 years ago

dhh1128 commented 2 years ago

Signed-off-by: dhh1128 daniel.hardman@gmail.com

dhh1128 commented 2 years ago

Fixes #219 Fixes #236

TelegramSam commented 2 years ago

Discussed WG 20220509. Soften MUST NOT to SHOULD NOT for authcrypt(sign(plaintext)) with a warning about anoncrypt(authcrypt(sign(plaintext))) not being valid, and must use anoncrypt(sign(plaintext)) being the thing to use in that case. No objections to merging this with proposed changes.

mccown commented 2 years ago

After Daniel made one minor change, this was approved in the DIDComm WG on 20220509.