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

Update firehose.mdx to explain authentication gotcha #316

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dom96
Copy link

@dom96 dom96 commented Jan 26, 2025

As me and other developers learned recently, the authentication provided by the Bluesky firehose is limited. My preference here would be to modify the firehose to avoid sending such events in the first place, but failing that I think that it's important to at least document this.

I do still consider myself a Bluesky noob, so feedback much appreciated if I've misused any of the terminology and/or if there is a clearer way to describe these concepts.

@yamarten
Copy link

yamarten commented Feb 2, 2025

IMHO, this may rather lead to misunderstanding. Your example doesn't seem to show "authentication is limited" but "authentication does not imply data validation". The listitem does not claim to be part of another person's repository, just refer to the list there.

In addition, this is not unique to firehose, so it is difficult to find a place to write the details. I recommend that you only explain that "authenticated" here means "signed".

Alternatively, if you want to draw attention to your example, you can add it to lists tutorial.

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

Successfully merging this pull request may close these issues.

2 participants