From b7c53d9a8035af452b54da67ee481509a5d33453 Mon Sep 17 00:00:00 2001 From: eidellev Date: Tue, 7 Mar 2023 11:52:28 +0200 Subject: [PATCH] fix: use normalized headers This ensures support for HTTP/2 thank you @KABBOUCHI ! --- providers/InertiaProvider/InertiaProvider.ts | 2 +- src/utils.ts | 12 ++++++------ test/validation.spec.ts | 1 + 3 files changed, 8 insertions(+), 7 deletions(-) diff --git a/providers/InertiaProvider/InertiaProvider.ts b/providers/InertiaProvider/InertiaProvider.ts index 866e061..10e7913 100644 --- a/providers/InertiaProvider/InertiaProvider.ts +++ b/providers/InertiaProvider/InertiaProvider.ts @@ -174,7 +174,7 @@ export default class InertiaProvider { Response.macro( 'redirect', function (path?: string, forwardQueryString: boolean = false, statusCode = 302): RedirectContract | void { - const isInertia = this.request.rawHeaders.includes(HEADERS.INERTIA_HEADER); + const isInertia = this.request.headers[HEADERS.INERTIA_HEADER]; const method = this.request.method; let finalStatusCode = statusCode; diff --git a/src/utils.ts b/src/utils.ts index d00456f..6aadcac 100644 --- a/src/utils.ts +++ b/src/utils.ts @@ -1,8 +1,8 @@ export enum HEADERS { - INERTIA_HEADER = 'X-Inertia', - INERTIA_PARTIAL_DATA = 'X-Inertia-Partial-Data', - INERTIA_PARTIAL_DATA_COMPONENT = 'X-Inertia-Partial-Component', - INERTIA_VERSION = 'X-Inertia-Version', - INERTIA_LOCATION = 'X-Inertia-Location', - VARY = 'Vary', + INERTIA_HEADER = 'x-inertia', + INERTIA_PARTIAL_DATA = 'x-inertia-partial-data', + INERTIA_PARTIAL_DATA_COMPONENT = 'x-inertia-partial-component', + INERTIA_VERSION = 'x-inertia-version', + INERTIA_LOCATION = 'x-inertia-location', + VARY = 'vary', } diff --git a/test/validation.spec.ts b/test/validation.spec.ts index 7ce7800..82ae61e 100644 --- a/test/validation.spec.ts +++ b/test/validation.spec.ts @@ -30,6 +30,7 @@ test.group('Validation negotiator', (group) => { }); } }); + test('Should use vanilla validator for HTML requests', async (assert) => { const app = await setup(); const ctx = app.container.use('Adonis/Core/HttpContext').create('/', {});