jazz icon indicating copy to clipboard operation
jazz copied to clipboard

Repro for the InvalidSignature issues on RNQuickCrypto

Open gdorsi opened this issue 7 months ago • 1 comments

Repro steps:

  • run cd examples/chat-rn-expo && pnpm install && pnpm ios to start the example app
  • if an account is already stored the loading should fail with an "InvalidSignature" error
  • otherwise load the chat co_zTD6yGYgLJf3gM72d7So4cgySb1
  • the loading should fail with an "InvalidSignature" error

gdorsi avatar May 06 '25 14:05 gdorsi

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
clerk-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
file-upload-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
form-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
gcmp-homepage ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
image-upload-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
jazz-chat ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
jazz-chat-1 ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
jazz-chat-2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
jazz-homepage ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
jazz-inspector ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
jazz-multi-cursors ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
jazz-organization ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
jazz-paper-scissors ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
jazz-pets ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
jazz-richtext ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
jazz-todo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
jazz-version-history ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
music-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
passkey-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
passphrase-auth-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
passwords-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm
reactions-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2025 3:24pm

vercel[bot] avatar May 06 '25 14:05 vercel[bot]

@gdorsi I rebased, added a commit that makes DX a bit better for certain XCode versions, but didn't make any headway.

After recent merges, I don't get the InvalidSignature error (maybe because you commented out my crap code in RNQuickCrypto). But I do get

 ERROR  Error withLoadedAccount {"err": [Error: Account has no profile]}

which seems to be from recent work on cojson/src/localNode.ts by you and @aeplay. Any way you can take a look at why we don't have the "default profile" behavior like we used to?

boorad avatar May 07 '25 22:05 boorad

@boorad Opened a PR to not throw when the profile is missing: https://github.com/garden-co/jazz/pull/2155

The profile is missing probably because it wasn't synced before destroying the node (it can happen if the node is immediately closed).

We can fix that, but for now I think we can just not throw if the profile is missing as it is not really needed anywhere by Jazz.

gdorsi avatar May 08 '25 08:05 gdorsi

Merged the fix here. Could you try again @boorad and see if everything works?

gdorsi avatar May 08 '25 13:05 gdorsi