fix(i18n): compute current locale from route instead of request

[ematipico]

Changes

Closes #9847
Closes PLT-1511

During the build, the url.pathname contains the the extension e.g. en.html, so Astro could never find the correct locale.

With this fix, we use RouteData instead, which is an object that we attach to the request using a pipeline hook. We attach this information here:

astro/packages/astro/src/core/build/generate.ts

Line 284 in f27f790

pipeline.onBeforeRenderRoute(i18nPipelineHook);

And this is the value of the function

astro/packages/astro/src/i18n/middleware.ts

Lines 137 to 139 in f27f790

	export const i18nPipelineHook: PipelineHookFunction = (ctx) => {
	Reflect.set(ctx.request, routeDataSymbol, ctx.route);
	};

Testing

I modified and existing test to render the current locale

Docs

N/A

[changeset-bot]

🦋 Changeset detected

Latest commit: 49d46e0

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