From 27bd9bfc8c6b6cbb0134160a14d0fb0904679616 Mon Sep 17 00:00:00 2001 From: sebas Date: Tue, 1 Oct 2024 09:56:19 -0600 Subject: [PATCH] nip46: fix HandleRequest request parsing logic Move out the `session.ParseRequest` call from the `else` branch. This caused an empty `Request` object to be used down in the function when the `Session` already existed. --- nip46/dynamic-signer.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/nip46/dynamic-signer.go b/nip46/dynamic-signer.go index a5ca4d4..7d5e1a5 100644 --- a/nip46/dynamic-signer.go +++ b/nip46/dynamic-signer.go @@ -105,11 +105,11 @@ func (p *DynamicSigner) HandleRequest(event *nostr.Event) ( } p.setSession(event.PubKey, session) + } - req, err = session.ParseRequest(event) - if err != nil { - return req, resp, eventResponse, fmt.Errorf("error parsing request: %w", err) - } + req, err = session.ParseRequest(event) + if err != nil { + return req, resp, eventResponse, fmt.Errorf("error parsing request: %w", err) } var secret string