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

getting python error #13

Closed
Funman2 opened this issue Jun 13, 2024 · 7 comments · Fixed by #14
Closed

getting python error #13

Funman2 opened this issue Jun 13, 2024 · 7 comments · Fixed by #14

Comments

@Funman2
Copy link

Funman2 commented Jun 13, 2024

hey,I tried to upgrade to your latest version of phoenixd-lnurl but with new version running from container ( in Portainer) i always get the error: ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

TypeError: ForwardRef._evaluate() missing 1 required keyword-only argument: 'recursive_guard'
seems to be some python version error, any idea?
_Phoenixd-lnurl_logs(1).txt

@AngusP
Copy link
Owner

AngusP commented Jun 14, 2024

Hey, just to check, are you using the image ghcr.io/angusp/phoenixd-lnurl:latest?

But yeah that does look like some issue with a package has appeared, I'll see if I can replicate the issue and resolve it quickly for you -- should hopefully just need some package updates

@Funman2
Copy link
Author

Funman2 commented Jun 14, 2024

Yes I tried that package and also all other packages since last month fail.The old package from one month ago still works. So went back.

@Funman2
Copy link
Author

Funman2 commented Jun 14, 2024

FYI running the container in portainer app on umbrel home server( portainer version 2.19.1) if that info herzlos helps to reproduce...

@AngusP
Copy link
Owner

AngusP commented Jun 14, 2024

Hrm, seems like it might be an issue specifically with pydantic (a dependency) and Pyhton 3.12 https://github.com/pydantic/pydantic-core/pull/1299

"Just" downgrading to Python 3.11 for now seems to fix it, I'm stuck on an older version of Pydantic for now to maintain compatibility with the lnurl package -- I'll merge a fix soon and then the :latest Docker image should work again

@AngusP
Copy link
Owner

AngusP commented Jun 14, 2024

@Funman2 thanks for the report, hopefully the :latest resolves for you (did for me!) but feel free to reopen if not 🤝

@Funman2
Copy link
Author

Funman2 commented Jun 14, 2024

Yes it works now, thanks very much for the really quick fix!
Does this packet version have some sort of nostr zap support already included?

@AngusP
Copy link
Owner

AngusP commented Jun 14, 2024

Does this packet version have some sort of nostr zap support already included?

Not yet, working on it in #11 -- it was blocked for a bit on an issue n phoenixd that's now been resolved (it would crash/hang on really big invoice descriptions, which is kinda what zaps are) so I just need to get around to finishing that pull request then we'll have proper zaps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants