From cefe9ddf348ed7fe470aca1dea105c71fd8c3033 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Francisco=20Louren=C3=A7o?= <208149+franciscolourenco@users.noreply.github.com> Date: Mon, 8 Jun 2020 21:21:23 +0200 Subject: [PATCH] fix!: Capitalize "Authorization" header name (#252) HTTP headers are case insensitive, however "Authorization" is a standard header name. Relevant documentation: - https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Authorization - https://tools.ietf.org/html/rfc7235#section-4.2 Co-authored-by: Guillaume Chau --- graphql-client/src/index.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/graphql-client/src/index.js b/graphql-client/src/index.js index a15c98c..ba4dd76 100644 --- a/graphql-client/src/index.js +++ b/graphql-client/src/index.js @@ -76,8 +76,8 @@ export function createApolloClient ({ // HTTP Auth header injection authLink = setContext(async (_, { headers }) => { - const authorization = await getAuth(tokenName) - const authorizationHeader = authorization ? { authorization } : {} + const Authorization = await getAuth(tokenName) + const authorizationHeader = Authorization ? { Authorization } : {} return { headers: { ...headers, @@ -122,8 +122,8 @@ export function createApolloClient ({ wsClient = new SubscriptionClient(wsEndpoint, { reconnect: true, connectionParams: () => { - const authorization = getAuth(tokenName) - return authorization ? { authorization, headers: { authorization } } : {} + const Authorization = getAuth(tokenName) + return Authorization ? { Authorization, headers: { Authorization } } : {} }, })