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.

Sign up for GitHub

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

Jump to bottom

Fix longstanding bug that breaks initial community view (fixes #3529) #4535

Merged

dessalines merged 1 commit into main from community-first-fetch

Mar 14, 2024

Member

Nutomic commented Mar 14, 2024

No description provided.


          Fix longstanding bug that breaks initial community view (fixes #3529)

0f3b883

Nutomic commented

View reviewed changes

crates/apub/src/api/list_posts.rs

-                  Some(resolve_actor_identifier::<ApubCommunity, Community>(name, &context, &None, true).await?)
-                    .map(|c| c.id)
+                  Some(
+                    resolve_actor_identifier::<ApubCommunity, Community>(name, &context, &local_user_view, true)

Member Author

Nutomic Mar 14, 2024

This is the problem, without auth it would only lookup in the local db and not make a webfinger request. No idea why it was hardcoded to None.

dessalines approved these changes

View reviewed changes

dessalines merged commit 835d329 into main

1 of 2 checks passed

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