-
Notifications
You must be signed in to change notification settings - Fork 2
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
Fetching Followers collection fails when a remote instance is down #169
Comments
Yeah we should detect failed accounts and delete them |
This is a temporarily out of order instance, so skipping then would be best. But my question is why do we need to issue requests to remote servers for locally available data? |
The data isn't cached locally yet. 😅 At least not actors outside our instance. |
Oh, the specific issue is we store the follower in the mention format instead of the actor URL so we need to convert on list. |
Is it difficult to store the different formats for actor addressing locally? Do they change in practice? |
We could do a database migration to link to actors instead of webfinger, yeah. It's not a trivial amount of work since we'd need to do a DB migration. We'd need to rework a bunch of validation logic too. |
can't we have a map stored separately though? |
Like a cache of webfinger to actor? That could be done, yes. Honestly we probably should have made the webfinger part optional and just resolved it to the actor url past the http route. 😅 |
This will be better once we have proper cache |
🎯 Success criteria: don't fail :P
Task Summary
Fetching the Follower list for distributed.press fails because an instance is down. Apparently it's trying to solve Webfinger for followers.
Related to:
The text was updated successfully, but these errors were encountered: