Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃悰 Fixed External Image URLs being incorrectly prefixed #20226

Merged
merged 3 commits into from
May 20, 2024

Conversation

ronaldlangeveld
Copy link
Member

@ronaldlangeveld ronaldlangeveld commented May 20, 2024

ref ENG-824

  • the bug is causing resize prefixes being added to images served from outside of Ghost.
  • this now would only append the prefex to images served by Ghost and other images urls' would get served as is.
  • we can determine that by checking whether imageName doesn't exist, meaning the source is a third party.
  • this mostly affect edge case users, eg where a feature image url was passed in via the API and doesn't get served by Ghost.

ref ENG-824

- the bug is causing resize prefixes being added to images served from outside of
  Ghost.
- this now would only append the prefex to images served by Ghost and
  other images urls' would get served as is.
- we can determine that by checking whether imageName doesn't exist,
  meaning the source is a third party.
- this mostly affect edge case users, eg where a feature image url was
  passed in via the API or Unsplash images.
@ronaldlangeveld ronaldlangeveld marked this pull request as ready for review May 20, 2024 10:01
@ronaldlangeveld ronaldlangeveld changed the title 馃悰 Fixed transform prefex appented to outside URLS 馃悰 Fixed Issue with External Image URLs being incorrectly prefixed May 20, 2024
@ronaldlangeveld ronaldlangeveld changed the title 馃悰 Fixed Issue with External Image URLs being incorrectly prefixed 馃悰 Fixed External Image URLs being incorrectly prefixed May 20, 2024
@ronaldlangeveld ronaldlangeveld merged commit e5056d8 into main May 20, 2024
22 checks passed
@ronaldlangeveld ronaldlangeveld deleted the fix-unintended-appending-of-transform branch May 20, 2024 10:06
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.

None yet

1 participant