Skip to content

Fix: changed cachet.hackclub.com to dunkirk.sh host since the images have not propogated#707

Open
jam06452 wants to merge 1 commit into
hackclub:mainfrom
jam06452:main
Open

Fix: changed cachet.hackclub.com to dunkirk.sh host since the images have not propogated#707
jam06452 wants to merge 1 commit into
hackclub:mainfrom
jam06452:main

Conversation

@jam06452

Copy link
Copy Markdown

Fix: Resolve default profile pictures by reverting CDN host to dunkirk.sh

This PR addresses a visual bug where user profile pictures (pfps) are displaying as the default green Slack avatars instead of the actual user images.
Root Cause

The caching CDN for Hack Club's Slack data is currently being onboarded to a new domain (https://cachet.hackclub.com). This new domain routes to a separate storage bucket where the profile photos have not yet fully propagated, causing the system to fall back to the default Slack avatars.

Reverted the host URL from cachet.hackclub.com back to the original https://cachet.dunkirk.sh. This routes the requests back to the Slack edge where the images are successfully cached, restoring the correct profile pictures while the new bucket finishes propagating.

This update modifies the base CDN URL string across our Ruby and JavaScript controllers. This change only affects the application at runtime when fetching images. Because the avatar URLs are constructed dynamically on-the-fly using the user's slack_id (e.g., https://cachet.dunkirk.sh/users/${user.slack_id}/r), it does not impact any build-time assets or static file compilation.

Files Changed:

app/controllers/search_controller.rb

app/javascript/controllers/mention_autocomplete_controller.js

app/models/user/profile.rb

@jam06452 jam06452 changed the title Fix: changed cachet.hackclub.com to dunkirk.sh host since the image Fix: changed cachet.hackclub.com to dunkirk.sh host since the images have not propogated Jun 26, 2026
@Lazylllama

Copy link
Copy Markdown
Member

This wouldn't have solved it in the long run, but its been a couple days without near as much downtime now anyways ✌🏻

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