at_client_sdk icon indicating copy to clipboard operation
at_client_sdk copied to clipboard

Published 20 hours ago • verified publisher icon atsign-foundation

Failed to get shared key while encrypting / decrypting message

Open nitesh2599 opened this issue 3 years ago • 8 comments

Describe the bug Getting the exception "Null" is not subtype of type 'FutureOr<String>' while sending and receiving notifications. We received a notification on atmosphere pro, we got the above mentioned exception while decrypting.

To Reproduce Steps to reproduce the behavior (Not always reproducible):

  1. Log in with perfectbluesnowskiing on the receiver side.
  2. Send a notification to the atsign.
  3. Got the above exception on decrypting.

Expected behavior The notification should be decrypted without any error.

Screenshots Screenshot 2022-05-26 at 3 20 10 PM Screenshot 2022-05-24 at 12 55 02 PM

Smartphone (please complete the following information):

  • Device: [e.g. iPhone11]
  • OS: [e.g. iOS14.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 59]

Were you using an @‎application when the bug was found?

  • @mosphere pro

Additional context Add any other context about the problem here.

nitesh2599 avatar May 26 '22 11:05 nitesh2599

I was able to replicate this issue only when the network is down. Steps I followed

  • Switched off the network
  • I could see the error in the app
  • Sent a file from a desktop app
  • Switched on the network back, I received the notification and was able to download the file.

https://user-images.githubusercontent.com/43639593/171405592-a394ed44-24c9-4c75-911c-a6d66dac5970.mp4

Note: In the desktop app, For the banamaniathenenvious atsign. There was an invalid or corrupted pad block exception.

Screenshot from 2022-06-01 17-51-27

purnimavenkatasubbu avatar Jun 01 '22 12:06 purnimavenkatasubbu

@nitesh2599 did you also face "invalid or corrupted pad block" exception as reported by Purnima?

murali-shris avatar Jun 02 '22 06:06 murali-shris

@murali-shris No, we didn't face "invalid or corrupted pad block".

nitesh2599 avatar Jun 03 '22 10:06 nitesh2599

@sachins-geekyants : This issue is fixed as a part of exception chaining PR and currently the changes are available in at_client trunk branch. Please re-test and let us know if you still see the issue.

sitaram-kalluri avatar Jun 07 '22 05:06 sitaram-kalluri

@sachins-geekyants can you confirm this has been fixed?

gkc avatar Jun 13 '22 10:06 gkc

@gkc I haven't experienced this issue lately, but we can keep this ticket open until a new version of atmosphere is available to test for wider users.

sachins-geekyants avatar Jun 13 '22 10:06 sachins-geekyants

We have deployed @mopsherePro 1.0.7(43), which uses shouldDecrypt parameter to decrypt notifications.

sachins-geekyants avatar Jun 20 '22 11:06 sachins-geekyants

Moving to requires validation.

sachins-geekyants avatar Jun 27 '22 09:06 sachins-geekyants

Closing as obsolete

gkc avatar Feb 20 '23 13:02 gkc