Skip to content

Allow the authorization header name to vary - #8

Merged
ssshah86 merged 3 commits into
prisma-labs:masterfrom
AcrossTheCloud:master
Jul 1, 2018
Merged

Allow the authorization header name to vary#8
ssshah86 merged 3 commits into
prisma-labs:masterfrom
AcrossTheCloud:master

Conversation

@matthewberryman

Copy link
Copy Markdown
Contributor

AWS AppSync uses x-api-key instead of Authorization, so I have a need to vary this. I've left it in such a way that 'Authorization' is the default if not specified, to keep this change backwards compatible.

@raduflp

Copy link
Copy Markdown

I was having the same issue. Would love to see this merged.

@ssshah86
ssshah86 merged commit aaa67f3 into prisma-labs:masterJul 1, 2018
@ssshah86

Copy link
Copy Markdown
Collaborator

Thanks for your contribution! Will upload to pypi shortly.

@matthewberryman

Copy link
Copy Markdown
ContributorAuthor

No worries. Thanks for a clean and easy to use library.

alaptseu pushed a commit to alaptseu/python-graphql-client that referenced this pull request Jul 18, 2018
Allow the authorization header name to vary
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@matthewberryman@raduflp@ssshah86