highperfocused/grimoire
1
0
Fork 0
mirror of https://github.com/purrgrammer/grimoire.git synced 2026-04-10 15:36:53 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix: establish WebSocket connection before showing QR code

Browse Source 
Fixes NIP-46 QR code login by opening the relay connections BEFORE
displaying the QR code. This ensures the client is listening when
the remote signer responds.

Also adds console logging for debugging NIP-46 connection flow.
This commit is contained in:
Claude
2026-01-12 10:19:43 +00:00
parent 27de57652e
commit 0b5634a75e
1 changed files with 10 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/components/nostr/LoginDialog.tsx
View File

@@ -175,11 +175,18 @@ export default function LoginDialog({ open, onOpenChange }: LoginDialogProps) {
});
signerRef.current = signer;
// IMPORTANT: Open the connection FIRST before showing QR
// This ensures we're listening when the signer responds
await signer.open();
// Generate the nostrconnect:// URI
const uri = signer.getNostrConnectURI({
name: "Grimoire",
url: window.location.origin,
});
// Log for debugging
console.log("[NIP-46] Generated nostrconnect URI:", uri);
setConnectUri(uri);
// Generate QR code with extra margin for better scanning
@@ -193,19 +200,19 @@ export default function LoginDialog({ open, onOpenChange }: LoginDialogProps) {
});
setQrDataUrl(dataUrl);
// Open the signer to start listening
await signer.open();
// Set up abort controller for cancellation
abortControllerRef.current = new AbortController();
setLoading(false);
// Wait for the remote signer to connect
console.log("[NIP-46] Waiting for remote signer...");
await signer.waitForSigner(abortControllerRef.current.signal);
console.log("[NIP-46] Remote signer connected!");
// Get the user's pubkey
const pubkey = await signer.getPublicKey();
console.log("[NIP-46] Got pubkey:", pubkey);
const account = new NostrConnectAccount(pubkey, signer);
handleSuccess(account);