CTemplar / webclient

Angular webclient (with Linux, macOS and Windows desktop clients) for CTemplar's encrypted email service.
https://ctemplar.com
Apache License 2.0
105 stars 28 forks source link

Error: Failed to get public keys #1564

Closed totallypredictable closed 2 years ago

totallypredictable commented 2 years ago

Your environment

Steps to reproduce

  1. Go to webclient -> contacts,
  2. Edit an existing contact (or create new contact),
  3. Click save

Expected behavior

Contact updated/saved without giving any errors.

Actual behavior

Getting an error saying "failed to get public keys"

Note: This issue only persists when my contacts are encrypted. It never used to happen until today. I was able to exchange pgp encrypted e-mails with my non-cTemplar contacts through automatic public key attachment and e-mail signing. It does not work today. Even though I have my contact's public key saved in their contact book on my mail.

Edit: Also, although I can send pgp-encrypted message to my contact because I had saved his public key in the contact book already, cTemplar does not attach my public-key to the e-mails that I send anymore. Agan, this never used to happen. It worked just fine until today. Public-key attachment option is activated both on my settings (Signature&Aliases -> click edit on my e-mail alias -> enable both "Attach Public Key" and "Sign messages") and on composer.

Recorded video of the issue:

https://user-images.githubusercontent.com/63866086/147690000-fddaad3e-ec10-47bb-9072-d34e2513b41c.mp4

OneWhiteBird commented 2 years ago

Confirmed on my end! Unable to save contact when "Contacts encryption" is enabled.

The-Hidden-Hand commented 2 years ago

Webclient is making a post to /keys/ endpoint, not sure why:

{"emails":[null]}

Related: https://sentry.io/organizations/ctemplar/issues/2912721702/?project=5256284&referrer=webhooks_plugin

The-Hidden-Hand commented 2 years ago

This is being actively solved as part of another bigger task. All the functions are being reviewed from the bottom up.