another isSafeMethod usage

elastic · Mar 3, 2020 · a5c12a8 · a5c12a8
1 parent e7829d9
commit a5c12a8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/core/server/http/lifecycle_handlers.ts b/src/core/server/http/lifecycle_handlers.ts
@@ -20,6 +20,7 @@
 import { OnPostAuthHandler } from './lifecycle/on_post_auth';
 import { OnPreResponseHandler } from './lifecycle/on_pre_response';
 import { HttpConfig } from './http_config';
+import { isSafeMethod } from './router';
 import { Env } from '../config';
 import { LifecycleRegistrar } from './http_server';
 
@@ -39,11 +40,10 @@ export const createXsrfPostAuthHandler = (config: HttpConfig): OnPostAuthHandler
       return toolkit.next();
     }
 
-    const isSafeMethod = request.route.method === 'get' || request.route.method === 'head';
     const hasVersionHeader = VERSION_HEADER in request.headers;
     const hasXsrfHeader = XSRF_HEADER in request.headers;
 
-    if (!isSafeMethod && !hasVersionHeader && !hasXsrfHeader) {
+    if (!isSafeMethod(request.route.method) && !hasVersionHeader && !hasXsrfHeader) {
       return response.badRequest({ body: `Request must contain a ${XSRF_HEADER} header.` });
     }