Skip to content

fix: remove images from external sources#486

Merged
ItsNeil17 merged 1 commit intomainfrom
fix/remove-external-images
Mar 17, 2026
Merged

fix: remove images from external sources#486
ItsNeil17 merged 1 commit intomainfrom
fix/remove-external-images

Conversation

@okunamayanad
Copy link
Copy Markdown
Member

@okunamayanad okunamayanad commented Mar 17, 2026

Pull Request

Description

Removes images from external sources.
These images are from sources, uncontrolled by HytaleModding. Currently they're blocking builds because Next can't fetch images and their sizes as the source site is down but in general, external images pose a security risk for the website as it gives control over content displayed on pages to a third party without first being reviewed.

Images should be added back in a future PR with being hosted by HM, this one is getting rid of them ASAP to get rid of the security risk and unblock builds.

Type of Change

  • Documentation fix (typo, grammar, clarification)
  • New documentation (guide, tutorial, page)
  • Bug fix
  • New feature
  • Other

Screenshots

If applicable, add before/after screenshots:

Checklist

  • Tested locally with bun run dev
  • Formatted code to adhere Styleguide with bun format
  • Ran bun audit (no critical vulnerabilities)
  • Checked spelling and grammar
  • Verified all links work
  • Followed Contributing Guidelines

Thank you for contributing!

@ItsNeil17 ItsNeil17 merged commit 8bceec5 into main Mar 17, 2026
5 of 6 checks passed
@ItsNeil17 ItsNeil17 deleted the fix/remove-external-images branch March 17, 2026 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants