Error: e.payment_hash is null

[KainerW]

Is there an existing issue for this?

• 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]

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

[reneaaron]

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

[dienummer]

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

[reneaaron]

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

[dienummer]

@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).

[reneaaron]

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

[pavanjoshi914]

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 #3074

[reneaaron]

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.