Skip to content

Request: Add cdn/workflow.serilum.com to allowedHostnames whitelist to bypass wsrv.nl caching#6567

Merged
Prospector merged 3 commits into
modrinth:mainfrom
ricksouth:patch-2
Jul 4, 2026
Merged

Request: Add cdn/workflow.serilum.com to allowedHostnames whitelist to bypass wsrv.nl caching#6567
Prospector merged 3 commits into
modrinth:mainfrom
ricksouth:patch-2

Conversation

@ricksouth

Copy link
Copy Markdown
Contributor

I've shortly talked with Prospector about this on Discord.

workflow.serilum.com is used for my badges:
image

They're automatically updated via a GitHub actions runner. But even though wsrv says they cache for a maximum of 30 days, it's been behind for longer than that. The url itself is a static website, ran through Cloudflare Pages. Which in itself is heavily cached and should have no negative effect on page load or user experience.

It would be great to bypass the wsrv conversion in descriptions for that domain. Similar to the other badge domains already in the whitelist.

Adding cdn.serilum.com is an alternative request to short-term fix the issue I'm having with wsrv not allowing webp's with more than 400 frames. I've made an issue regarding that here:
#6566

That domain is also just a Cloudflare Pages static website. And identically cached.

Let me know if I need to change anything on my end for this! :)

Signed-off-by: Rick South <rick@natamus.com>
@modrinth-bot

modrinth-bot commented Jun 30, 2026

Copy link
Copy Markdown
Member

Note

This changelog has been baked. Any further edits will not be reflected.

Pull request changelog

App

Added

Changed

  • Added cdn.serilum.com and workflow.serilum.com as domains that can bypass the image proxy.

Deprecated

Removed

Fixed

Security

Website

Added

Changed

  • Added cdn.serilum.com and workflow.serilum.com as domains that can bypass the image proxy.

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@Prospector Prospector enabled auto-merge July 4, 2026 21:30
Signed-off-by: Prospector <6166773+Prospector@users.noreply.github.com>
Signed-off-by: Prospector <6166773+Prospector@users.noreply.github.com>
@Prospector Prospector added this pull request to the merge queue Jul 4, 2026
Merged via the queue into modrinth:main with commit a2adbe1 Jul 4, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants