lightning-browser-extension icon indicating copy to clipboard operation
lightning-browser-extension copied to clipboard

Error: e.payment_hash is null

Open KainerW opened this issue 1 year ago • 8 comments
trafficstars

Is there an existing issue for this?

  • [X] I have searched the existing issues

Describe the bug

Opened up my browser extension today and it shows "Error: e.payment_hash is null" at the bottom.

What could have happenend?

Screenshots [optional]

Screenshot at Dec 12 11-07-20

Steps To Reproduce

Just look into your browser extension and see it at the bottom.

Expected behavior

I expect that mesage to just not be there and my recent transaction loading and showing.

Alby information

3.4.1

Installed to browser via extensions mozilla/firefox.

Device information

firefox 120.0.1

Additional context

No response

Are you working on this?

No

KainerW avatar Dec 12 '23 10:12 KainerW

@KainerW Thanks for reporting. Could you share which connector you are using?

reneaaron avatar Dec 18 '23 09:12 reneaaron

Happens to me with LNDHub

I had to nuke my entire LNDHub redis database to make it stop. Probably relates to expired unpaid invoices.

I just wanted to use nostr TBH

dienummer avatar Jan 30 '24 04:01 dienummer

@dienummer Can you share some details about what LNDHub you are running? lndhub.go?

reneaaron avatar Jan 30 '24 13:01 reneaaron

@dienummer Can you share some details about what LNDHub you are running? lndhub.go?

https://github.com/BlueWallet/LndHub

Not gonna lie, I didn't know about lndhub.go (or maybe vaguely think I remember hearing about it, but forgot).

dienummer avatar Feb 27 '24 22:02 dienummer

@pavanjoshi914 Can you maybe look into this? Could we keep the field paymentHash optional for the LNDHub connector?

reneaaron avatar Mar 06 '24 17:03 reneaaron

as far as i understand this error shall be thrown by getTransactions action.

null values are returned when payment_hash field is not present in the object itself. but we create responses returned from the connector manually. so payment_hash is always there to be consumed by getTransactions action

i also tried assigning null value to payment_hash and try but still worked for me. no error. still here is the fix https://github.com/getAlby/lightning-browser-extension/pull/3074

pavanjoshi914 avatar Mar 11 '24 05:03 pavanjoshi914

Well then let's first try to understand the underlying problem. I don't think we just want to throw code on unclear problems.

According to the docs this field should always be there, too: https://github.com/BlueWallet/LndHub/blob/master/doc/Send-requirements.md#get-getuserinvoices

@dienummer Does this problem still persist for you after nuking your redis? How do you come to the conclusion it's about expired unpaid invoices? Any info would be really helpful to reproduce this problem.

reneaaron avatar Mar 13 '24 13:03 reneaaron