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

Fetching Followers collection fails when a remote instance is down #169

Closed
fauno opened this issue May 16, 2024 · 9 comments
Closed

Fetching Followers collection fails when a remote instance is down #169

fauno opened this issue May 16, 2024 · 9 comments
Labels
bug / not working Something isn't working review needed This issue is ready for testing

Comments

@fauno
Copy link
Collaborator

fauno commented May 16, 2024

🎯 Success criteria: don't fail :P

Task Summary

image

Fetching the Follower list for distributed.press fails because an instance is down. Apparently it's trying to solve Webfinger for followers.

Related to:

@RangerMauve
Copy link
Contributor

Yeah we should detect failed accounts and delete them

@fauno
Copy link
Collaborator Author

fauno commented May 16, 2024

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?

@RangerMauve
Copy link
Contributor

The data isn't cached locally yet. 😅 At least not actors outside our instance.

@RangerMauve
Copy link
Contributor

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.

@fauno
Copy link
Collaborator Author

fauno commented May 21, 2024

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?

@RangerMauve
Copy link
Contributor

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.

@sutty-coop sutty-coop added the review needed This issue is ready for testing label Jun 4, 2024
@sutty-coop sutty-coop moved this from Needs triage to In Progress in Distributed Press Organizing Jun 4, 2024
@fauno
Copy link
Collaborator Author

fauno commented Jul 8, 2024

can't we have a map stored separately though?

@RangerMauve
Copy link
Contributor

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. 😅

@sutty-coop
Copy link
Collaborator

This will be better once we have proper cache

@github-project-automation github-project-automation bot moved this from In Progress to Done in Distributed Press Organizing Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug / not working Something isn't working review needed This issue is ready for testing
Projects
Status: Done
Development

No branches or pull requests

3 participants