From f36dee28789b1187652874fe76d1801c9a8e5974 Mon Sep 17 00:00:00 2001 From: Jake Champion Date: Thu, 10 Mar 2022 09:39:15 +0000 Subject: [PATCH] increase max uri length to 8k to match the limit documented on https://docs.fastly.com/en/guides/resource-limits (#71) At time of writing, the maximum uri allowed on fastly is 8kb --- c-dependencies/js-compute-runtime/js-compute-builtins.cpp | 2 +- c-dependencies/xqd.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/c-dependencies/js-compute-runtime/js-compute-builtins.cpp b/c-dependencies/js-compute-runtime/js-compute-builtins.cpp index 2db54807d3..5f7dbcb789 100644 --- a/c-dependencies/js-compute-runtime/js-compute-builtins.cpp +++ b/c-dependencies/js-compute-runtime/js-compute-builtins.cpp @@ -243,7 +243,7 @@ inline bool ReturnPromiseRejectedWithPendingError(JSContext* cx, const JS::CallA return true; } -#define HANDLE_READ_CHUNK_SIZE 1024 +#define HANDLE_READ_CHUNK_SIZE 8192 template static char* read_from_handle_all(JSContext* cx, HandleType handle, diff --git a/c-dependencies/xqd.h b/c-dependencies/xqd.h index 567e39eb33..b75609a5cb 100644 --- a/c-dependencies/xqd.h +++ b/c-dependencies/xqd.h @@ -15,7 +15,7 @@ extern "C" { //max header size to match vcl #define HEADER_MAX_LEN 69000 #define METHOD_MAX_LEN 1024 -#define URI_MAX_LEN 4096 +#define URI_MAX_LEN 8192 #define DICTIONARY_ENTRY_MAX_LEN 8000 // TODO ACF 2020-01-17: these aren't very C-friendly names