Skip to content

Commit

Permalink
proceed as usual after warning
Browse files Browse the repository at this point in the history
  • Loading branch information
lilnasy committed Dec 6, 2023
1 parent 61b5c17 commit 15420ce
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion packages/astro/src/core/routing/manifest/create.ts
Original file line number Diff line number Diff line change
Expand Up @@ -460,7 +460,7 @@ export function createRouteManifest(
destination = to.destination
}
if (destination.startsWith("http")) {
return logger.error('redirects', `Redirecting to an external URL is not supported: ${from} -> ${to}`);
logger.warn('redirects', `Redirecting to an external URL is not officially supported: ${from} -> ${to}`);
}
}

Expand Down
3 changes: 2 additions & 1 deletion packages/astro/test/redirects.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,8 @@ describe('Astro.redirect', () => {
expect(response.headers.get('location')).to.equal('/login');
});

it('Ignores external redirect', async () => {
// ref: https://github.com/withastro/astro/pull/9287
it.skip('Ignores external redirect', async () => {
const app = await fixture.loadTestAdapterApp();
const request = new Request('http://example.com/external/redirect');
const response = await app.render(request);
Expand Down

0 comments on commit 15420ce

Please sign in to comment.