Replying to Avatar The Daniel 🖖

Very helpful, thanks! It does appear that there's a bug in nos2x that prevents it from properly decrypting wallet backups using the NIP44 method. This is something we’ll have to catch so we can display a more helpful error message.
Avatar
fiatjaf 1d ago 💬 1

The ciphertext being decrypted in his screenshot is a NIP-04 one, notice it ends with an "?iv=..." parameter.

Reply to this note

Please Login to reply.

Discussion

Avatar
The Daniel 🖖 1d ago 💬 1

Thanks for catching that. This will help improve the detection logic in these random cases.

Avatar
fiatjaf 1d ago 💬 1

What I'm saying is that the app that created this backup encrypted the seed with NIP-04 instead of NIP-44. It was a bug there. Just fix that, don't go around trying to detect edge cases.

Avatar
The Daniel 🖖 1d ago

Right, what I mean by edge case is that the app that created the backup was an earlier proof-of-concept that few people except myself and nostr:npub1vxd0dfst8ljvwva2egrpc53ve8ru78v8aaxfpravchkexmfmmu3sqnrs50 have used before, meaning the chance of other users experiencing this issue is low. It allowed both encryption methods because one of the signer extensions tested did not support NIP44, so our import function needs just to detect this. That's what this fix will cover.