Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invoice Dashboard fails to display *any* requests #327

Closed
MantisClone opened this issue Jan 23, 2025 · 3 comments · Fixed by #337 · May be fixed by RequestNetwork/requestNetwork#1569
Closed

Invoice Dashboard fails to display *any* requests #327

MantisClone opened this issue Jan 23, 2025 · 3 comments · Fixed by #337 · May be fixed by RequestNetwork/requestNetwork#1569
Assignees

Comments

@MantisClone
Copy link
Member

MantisClone commented Jan 23, 2025

Problem

Invoice Dashboard fails to display any requests

Screen.Recording.2025-01-23.at.16.14.27.mov
@MantisClone MantisClone transferred this issue from RequestNetwork/requestNetwork Jan 23, 2025
@MantisClone
Copy link
Member Author

Maybe we're not sending the domain to lit protocol correctly. Or maybe it's a bug in the Lit SDK. Unclear.

@rodrigopavezi
Copy link
Member

I could replicate this issue when Lit Session has expired. So user need to disconnect wallet and connect wallet again so the Lit Sign message will apear and allow decryption to work again.

@rodrigopavezi
Copy link
Member

After further investigation it looks like the issue is happening on the LIT SDK.

Message I sent to Lit support:

I think I found a bug in the Lit SDK and I want to see if that is the case.

We are getting an domain undefined error in our application when the session expires. So as we discussed before here we are passing our application domain to the createSiweMessage function.

The thing is that this function is called inside the authNeededCallback

So when our application calls it at the first time everything works as we pass the domain param when calling our application get session function.

But when LIT calls authNeededCallback later on, the domain param will be undefined

It looks like LIT SDK should add domain param to the AuthCallbackParams

Then when authNeededCallback is called again because the session has expired it would have the correct domain saved in there like the uri param.

See image bellow

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment