Support Aegis URL scheme for NIP-46 login #161

2026-04-06T07:33:41Z

priyanshu_bharti commented

2026-04-06 07:33:41 +00:00

Summary

Adds support for Aegis and nostrsigner x-callback URL launch flow for NIP-46 login.
Handles signer callback URLs safely so the app does not navigate to callback paths.
Shows a clear error toast when signer returns an auth error.

What changed

Added x-callback URL generation for connect flow with these params:
method=connect
nostrconnect
x-source
x-success
x-error
Added remote signer launch actions in the QR login flow.
Added callback interception for authSuccess and authError in app URL open handling.

Testing done

pnpm run check passed.
pnpm run lint passed.
Local web sanity test passed:
Remote signer QR screen opens correctly.
QR is generated correctly.
Clicking QR copies the value and shows Copied to clipboard toast.

Issue

Ref #83

## Summary Adds support for Aegis and nostrsigner x-callback URL launch flow for NIP-46 login. Handles signer callback URLs safely so the app does not navigate to callback paths. Shows a clear error toast when signer returns an auth error. ## What changed Added x-callback URL generation for connect flow with these params: method=connect nostrconnect x-source x-success x-error Added remote signer launch actions in the QR login flow. Added callback interception for authSuccess and authError in app URL open handling. ## Testing done pnpm run check passed. pnpm run lint passed. Local web sanity test passed: Remote signer QR screen opens correctly. QR is generated correctly. Clicking QR copies the value and shows Copied to clipboard toast. ![image.png](/attachments/a48f129f-db04-4363-a254-9370b53a17b1) Issue Ref #83

image.png

37 KiB

priyanshu_bharti commented

2026-04-06 17:10:59 +00:00

This is now fixed and working properly ✅
@hodlbod ,please take a look and merge if it's good.
Thanks!

This is now fixed and working properly ✅ @hodlbod ,please take a look and merge if it's good. Thanks!

hodlbod requested changes 2026-04-06 19:00:26 +00:00

src/app/components/BunkerConnect.svelte Outdated

						
				@@ -22,1 +32,4 @@

				      <p class="text-sm opacity-75">Scan with your signer to log in, or click to copy.</p>

				      {#if Capacitor.isNativePlatform()}

				        <div class="flex w-full flex-col gap-2">

				          <Button class="btn btn-primary" onclick={openAegis}>Open in Aegis</Button>

hodlbod commented

2026-04-06 18:30:56 +00:00

Check if this platform is specifically ios, and only show "Open in Signer", not "Open in Aegis" (the aegis:// scheme I think is for backwards compatibility, which means we only need to deal with the nostrsigner:// scheme).

priyanshu_bharti marked this conversation as resolved

src/app/util/nip46.ts Outdated

						
				@@ -36,0 +56,4 @@

				    this.signerUrls.set({

				      aegis: makeSignerLaunchUrl("aegis", url),

				      nostrsigner: makeSignerLaunchUrl("nostrsigner", url),

				    })

hodlbod commented

2026-04-06 18:32:05 +00:00

This seems like unnecessary state/indirection

priyanshu_bharti marked this conversation as resolved

priyanshu_bharti commented

2026-04-10 13:57:08 +00:00

hii @hodlbod, could you please review it when you have time. Thanks !
if needed any type of changes,please let me know

hii @hodlbod, could you please review it when you have time. Thanks ! if needed any type of changes,please let me know

priyanshu_bharti requested review from hodlbod 2026-04-10 14:28:28 +00:00

hodlbod requested changes 2026-04-10 16:28:30 +00:00

src/app/components/BunkerConnect.svelte Outdated

						
				@@ -23,0 +31,4 @@

				        <div class="flex w-full flex-col gap-2">

				          <Button class="btn btn-primary" onclick={openSigner}>Open in Signer</Button>

				        </div>

				      {/if}