Skip to content

Commit

Permalink
increase max uri length to 8k to match the limit documented on https:…
Browse files Browse the repository at this point in the history
…//docs.fastly.com/en/guides/resource-limits (#71)

At time of writing, the maximum uri allowed on fastly is 8kb
  • Loading branch information
JakeChampion committed Mar 10, 2022
1 parent a5ebea9 commit f36dee2
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion c-dependencies/js-compute-runtime/js-compute-builtins.cpp
Expand Up @@ -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<auto op, class HandleType>
static char* read_from_handle_all(JSContext* cx, HandleType handle,
Expand Down
2 changes: 1 addition & 1 deletion c-dependencies/xqd.h
Expand Up @@ -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
Expand Down

0 comments on commit f36dee2

Please sign in to comment.