Skip to content

docs: fix broken case study logos - #1371

Open
prx-my wants to merge 3 commits into
OWASP:mainfrom
prx-my:docs/fix-case-study-logos
Open

prx-my wants to merge 3 commits into
OWASP:mainfrom
prx-my:docs/fix-case-study-logos

Conversation

@prx-my

@prx-my prx-my commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

What changed and why

The Analog and Ghost case studies render a broken logo because both upstream raw GitHub paths now return 404.

Rather than swap one hotlink for another, this vendors both logos into the site's own static images folder and references them locally, matching how the Storybook, Mastra, VS Code, and n8n logos are handled:

  • website/static/img/analog-logo.svg
  • website/static/img/ghost-logo.png
  • website/docs/case-studies/analog.md
  • website/docs/case-studies/ghost.md

Closes #1376

@prx-my
prx-my requested a review from sonukapoor as a code owner October 21, 2026 19:27
Copilot AI balanced review requested due to automatic review settings October 21, 2026 19:27

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@sonukapoor sonukapoor left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The report is right and so is the diagnosis. I fetched all four URLs: both of the current ones return 404 and both of yours return 200, so this does restore the images.

What I would rather do is fix the class instead of the instance. We already keep four case-study logos in the site's own static images folder, Storybook, Mastra, VS Code and n8n, and the two that just broke are broken precisely because they were hotlinked instead. Swapping one hotlink for another leaves them one upstream site reshuffle away from breaking again, and on a page we point journalists and adopters at.

So: download those two files into the static images folder and reference them locally, the same way the Storybook one is done. Same two lines of change, and it stops being our problem. Thanks for catching this, broken images on the case studies are worse than they look given who reads those pages.

Comment thread website/docs/case-studies/analog.md Outdated
@prx-my
prx-my force-pushed the docs/fix-case-study-logos branch from 785530c to 52d909e Compare October 22, 2026 09:00
@prx-my

prx-my commented Oct 22, 2026

Copy link
Copy Markdown
Contributor Author

Done — both logos are now vendored into website/static/img and referenced locally, the same way Storybook, Mastra, VS Code, and n8n are handled. No hotlinks left in these two case studies.

@prx-my prx-my closed this Oct 22, 2026
@prx-my prx-my reopened this Oct 22, 2026

@sonukapoor sonukapoor left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the version I wanted, thank you. Both files are vendored now and referenced with the site base path in front, which is exactly how the other eighteen case-study logos are done.

I pulled both off your branch to be sure they are not empty placeholders: the Analog one is a real 9.7KB SVG and the Ghost one is a 400 by 400 PNG at 44KB. Both upstream URLs are still returning 404, so this does fix the pages rather than moving the breakage.

Approved.

This branch has not been deployed

No deployments
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.

[Docs] Analog and Ghost case study logos are broken (404)

3 participants