Skip to content

Fix relative redirects regression. Closes #13607#13616

Merged
ematipico merged 3 commits intowithastro:mainfrom
lfilho:fix-local-relative-redirects
Apr 16, 2025
Merged

Fix relative redirects regression. Closes #13607#13616
ematipico merged 3 commits intowithastro:mainfrom
lfilho:fix-local-relative-redirects

Conversation

@lfilho
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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?

Comment thread .changeset/tasty-cities-warn.md Outdated
Comment thread .changeset/tasty-cities-warn.md Outdated
Comment thread packages/astro/test/redirects.test.js Outdated
Comment thread packages/astro/test/redirects.test.js Outdated
@lfilho
Copy link
Copy Markdown
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
Co-authored-by: Emanuele Stoppa <my.burning@gmail.com>
Co-authored-by: Emanuele Stoppa <my.burning@gmail.com>
@lfilho
Copy link
Copy Markdown
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
…13616)

Co-authored-by: Emanuele Stoppa <my.burning@gmail.com>
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