Skip to content

Conversation

@lfilho
Copy link
Contributor

@lfilho lfilho commented Apr 15, 2025

Changes

As per #13607, redirects had a regression with relative redirects introduced in #12979. This PR fixes it.

Fixes #13607

Testing

  • Unit tests
  • Tested with my local side project using the new changes

Docs

/cc @withastro/maintainers-docs for feedback! This was never documented but always worked since early versions of Astro so as mentioned by @ematipico on #13607 we should probaly update the docs. Happy to have a stab at it if you want too.

@changeset-bot
Copy link

changeset-bot bot commented Apr 15, 2025

🦋 Changeset detected

Latest commit: c096879

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added pkg: astro Related to the core `astro` package (scope) semver: minor Change triggers a `minor` release labels Apr 15, 2025
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This PR is blocked because it contains a minor changeset. A reviewer will merge this at the next release if approved.

@lfilho lfilho force-pushed the fix-local-relative-redirects branch 2 times, most recently from 3fbad8f to 92a43e2 Compare April 15, 2025 03:06
@lfilho lfilho changed the title Fix relative redirects regression Fix relative redirects regression. Closes #13607 Apr 15, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Apr 15, 2025

CodSpeed Performance Report

Merging #13616 will not alter performance

Comparing lfilho:fix-local-relative-redirects (02c5059) with main (eda0fb5)

Summary

✅ 6 untouched benchmarks

Copy link
Member

@ematipico ematipico left a comment

Choose a reason for hiding this comment

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

Thank you @lfilho for you contribution. I believe the tests are incorrect, can you please update them?

@lfilho
Copy link
Contributor Author

lfilho commented Apr 15, 2025

Oh 🤦 forgive me, shouldn't have done it after a long day at work 😄 Funnily enough it worked and test passed 😂 I'll push a fix later today

@lfilho lfilho force-pushed the fix-local-relative-redirects branch from 92a43e2 to c85a46e Compare April 16, 2025 04:52
lfilho and others added 2 commits April 15, 2025 21:54
@lfilho
Copy link
Contributor Author

lfilho commented Apr 16, 2025

@ematipico ready for you again

@lfilho lfilho requested a review from ematipico April 16, 2025 04:56
@ematipico ematipico merged commit d475afc into withastro:main Apr 16, 2025
2 checks passed
@astrobot-houston astrobot-houston mentioned this pull request Apr 16, 2025
openscript pushed a commit to openscript/astro that referenced this pull request Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope) semver: minor Change triggers a `minor` release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Regression bug with configuredRedirects: local relative redirects are being interpreted as external urls

2 participants