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

MissingField: Field "user_id" #18

Open
hanas879 opened this issue Jan 23, 2025 · 3 comments
Open

MissingField: Field "user_id" #18

hanas879 opened this issue Jan 23, 2025 · 3 comments

Comments

@hanas879
Copy link
Contributor

hanas879 commented Jan 23, 2025

I noticed that some episodes where missing on my server. When I checked the logs for pasjonsfrukt this error is present:

MissingField: Field "user_id" of type str is missing in SchibstedCredentials 
pasjonsfrukt  | instance
pasjonsfrukt  | Unclosed client session
pasjonsfrukt  | client_session: <aiohttp.client.ClientSession object at 0x7f5c2590dd90>
pasjonsfrukt  | Unclosed connector
pasjonsfrukt  | connections: ['[(<aiohttp.client_proto.ResponseHandler object at 0x7f5c25887bd0>, 8043863.669657797)]', '[(<aiohttp.client_proto.ResponseHandler object at 0x7f5c258ec590>, 8043864.934751507)]', '[(<aiohttp.client_proto.ResponseHandler object at 0x7f5c258866d0>, 8043865.007322781)]']
pasjonsfrukt  | connector: <aiohttp.connector.TCPConnector object at 0x7f5c2590e710>
@hanas879
Copy link
Contributor Author

Maybe they have updated the API? I'm not that into the API part so I haven't checked much, but there seems to be a call to user_id in the POST requests when trying to sign into my account from a browser.

@mathiazom
Copy link
Owner

mathiazom commented Jan 23, 2025

Nice catch!

This is from podme-api, so @bendikrb might have more insights here.

But yea, without testing myself it seems like the API response might have changed to not include user_id. If that is the case, we should remove this unused property, or at least make it optional.

@mathiazom
Copy link
Owner

mathiazom commented Jan 26, 2025

Looks like the problem is not just the user_id field. It's seems the refresh_token is also missing, which means we need to re-authorize each time. I have made a temporary fix in my fork of podme_api (basically disabled refreshing and credentials storage) and updated the canary branch of pasjonsfrukt. So for now you should be able to use the canary image

docker pull ghcr.io/mathiazom/pasjonsfrukt:canary

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

No branches or pull requests

2 participants