No checking of err result of AddInvoice in the lnurl block

bumi / lnme

Your friendly Bitcoin Lightning ⚡ payment page ⚡

MIT License

167 stars 30 forks source link

Closed joostjager closed 3 years ago

joostjager commented 3 years ago

Instead an empty invoice is returned in the pr field. Makes it hard to diagnose issues.

bumi commented 3 years ago

true, yes! LNURL needs better logging. Will add it.

Can I help you with something in the meantime? Does the normal new invoice endpoint work for you?

joostjager commented 3 years ago

Which one is the normal new invoice endpoint? If you mean request an invoice via the webpage, yes that works. I am running 1.3.0.

bumi commented 3 years ago

Yes, this one that means you only have the problem with lnurl/lightning address?

joostjager commented 3 years ago

Ah no, there was something wrong with my lnd and none of the endpoints worked. Now that that is fixed, both work.

bumi commented 3 years ago

yay, great. now you should also see a proper error message in both endpoints.

Let me know if you have some other issues or some ideas what should be improved for your needs.