Skip to content

Updates to issue purge requests to CDN - #854

Merged
bradchoate merged 1 commit into
masterfrom
purge-for-shake-user-image-updates
Sep 7, 2025
Merged

bradchoate merged 1 commit into
masterfrom
purge-for-shake-user-image-updates

Conversation

@bradchoate

Copy link
Copy Markdown
Collaborator

Particularly for saving shake and profile images. These have long TTLs like other assets served via CDN, but unlike post images, they can be updated and so have to be purged when they are updated.

Note that we don't really have a size limitation for uploaded profile, shake images. They are forcibly resized to 200x200 (or left alone if smaller than this). The site places no restriction on the resulting image size, since a 200x200 JPG should be pretty small.

Fixes #450 #447 #345 maybe others

Particularly for saving shake and profile images. These have long
TTLs like other assets served via CDN, but unlike post images, they
can be updated and so have to be purged when they are updated.

Note that we don't really have a size limitation for uploaded profile,
shake images. They are forcibly resized to 200x200 (or left alone
if smaller than this). The site places no restriction on the resulting
image size, since a 200x200 JPG should be pretty small.

Fixes #450 #447 #345 maybe others
@bradchoate
bradchoate merged commit b26dd73 into master Sep 7, 2025
4 checks passed
@bradchoate
bradchoate deleted the purge-for-shake-user-image-updates branch September 7, 2025 18:23
@jessamynwest

Copy link
Copy Markdown
Contributor

Yay thank you!!

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.

Shake Image does not update when new image file chosen

2 participants