Skip to content

Preconnect to go.bsky.app #8825

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

Merged
merged 2 commits into from
Aug 13, 2025
Merged

Preconnect to go.bsky.app #8825

merged 2 commits into from
Aug 13, 2025

Conversation

mozzius
Copy link
Member

@mozzius mozzius commented Aug 12, 2025

Might make the link proxy service a little bit faster?

Also, should we use preconnect or dns-prefetch here? I don't know

Copy link

github-actions bot commented Aug 12, 2025

Old size New size Diff
7.58 MB 7.58 MB 50 B (0.00%)

@surfdude29
Copy link
Contributor

surfdude29 commented Aug 12, 2025

I'm obviously no expert on these matters, but my understanding from reading things like this StackOverflow answer is that preconnect might be more helpful than dns-prefetch here, because it would also set up the TLS connection to go.bsky.app, which could be pretty useful in speeding up loading links.

However, would preconnect cause a lot of extra requests to go.bsky.app that end up being unnecessary, because the user doesn't click a link? That sounds like it would be the main consideration 🤔

Also, would it help to add either preconnect or dns-prefetch here as well?

<!--
Preconnect to essential domains
-->
<link rel="preconnect" href="https://bsky.social">
<title>%WEB_TITLE%</title>

@arcalinea arcalinea temporarily deployed to samuel/preconnect-gobskyapp - social-app PR #8825 August 12, 2025 12:08 — with Render Destroyed
@mozzius mozzius requested a review from haileyok August 12, 2025 12:08
@estrattonbailey estrattonbailey merged commit eef639e into main Aug 13, 2025
7 checks passed
@mozzius mozzius deleted the samuel/preconnect-gobskyapp branch August 13, 2025 22:04
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.

4 participants