Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

ngoue
Copy link
Contributor

@ngoue ngoue commented Apr 24, 2019

Add support for case-insensitive headers to the oauth1 BaseEndpoint._create_request to ensure headers are recognized in all endpoints.

@ngoue
Copy link
Contributor Author

ngoue commented Apr 24, 2019

I'm working with AWS API Gateway and right now I'm having to alter the Content-Type header manually in order to get my SignatureOnlyEndpoint to validate correctly.

Let me know if I'm using this wrong, but I figured we should be allowing case-insensitive headers here as well.

@JonathanHuot JonathanHuot added this to the 3.1.0 milestone Apr 26, 2019
@JonathanHuot
Copy link
Member

LGTM

@JonathanHuot
Copy link
Member

Hi @ngoue, could you please add a commit to retrigger the Travis-CI build ? I open an issue on their supprot because it didn't trigger any builds for your PR. Now they say it is fixed, but it requires another commit/push.

@JonathanHuot
Copy link
Member

Hi @ngoue, could you please retry again to force-push ? I'm liaising with Travis-CI support team to understand why this is again triggering issues.

@JonathanHuot
Copy link
Member

Hi @ngoue, it seems Travis CI team is not able to remove you from the "Abuse" list because you have no TravisCI account attached to your github account. I will run the build on locally on my side and if the tests are passing I'll force the merge of this PR.
Stay with me :-)

@JonathanHuot JonathanHuot merged commit c376e27 into oauthlib:master May 6, 2019
@JonathanHuot
Copy link
Member

Build has been run locally due to the TravisCI bug and all tests were passing.

@ngoue
Copy link
Contributor Author

ngoue commented May 6, 2019

Hahaha interesting... I guess I'll go make a Travis CI account and connect them. Sorry to cause so much trouble!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants