Repository navigation
Iterating over Liked private likes filters - #323
Merged
Merged
Conversation
aaronjae22
added this pull request to stack #321
October 5, 2026 21:14
lisad
approved these changes
Oct 6, 2026
| actor=actor, | ||
| timestamp=liked_at, | ||
| visibility="public", | ||
| visibility=VISIBILITIES[(number - 1) % len(VISIBILITIES)], |
Member
There was a problem hiding this comment.
A cool option for this idiom is to have the function or object own a couple cycles. From python 'itertools.cycle'
visibilities = cycle(VISIBILITIES)
for i in range(NOTE_COUNT):
...
visibility = next(visibility)
...
Collaborator
Author
There was a problem hiding this comment.
I agree, that would be much cleaner. I'll do it in the follow up PR
aaronjae22
force-pushed
the
liked-private-likes
branch
from
October 9, 2026 16:07
476a5c9 to
23ab6b6
Compare
aaronjae22
force-pushed
the
liked-private-likes
branch
from
October 9, 2026 16:25
23ab6b6 to
096947f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The liked collection was keeping only public Likes even though the endpoint already requires a portability token bound to the account. The filter was only doing one thing which was hiding an account's own private and followers-only Likes from that account's own token. Those Likes never reach the destination and nothing tells the user.
Alice has one public, one followers-only and one private Like, and fetches her liked collection with her own token and it was receiving
liked totalItems = 1 of 3LOLA §6.4 expects the source to provide the liked collection to the migration token "No changes should need to be made", and we were dropping data without warning anyone so this is a change on it.
The content collection and the outbox both give a bound token every visibility, the liked collection was the only one that didn't.
There's a second little isue. Signup provisioning made every Like public because of this filter. So a new account never had a non-public Like.
Current
liked_collectionfiltersvisibility="public"beneath the strict scope gate⠀After this PR