Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make sure received signatures headers are correctly cased #75

Closed
OGKevin opened this issue Nov 28, 2017 · 0 comments
Closed

Make sure received signatures headers are correctly cased #75

OGKevin opened this issue Nov 28, 2017 · 0 comments
Assignees
@OGKevin
Labels
enhancement
Milestone
0.12.4

Comments

@OGKevin
Copy link
Contributor

OGKevin commented Nov 28, 2017 •
edited

Steps to reproduce:

  1. Receive a response from the API
  2. The response headers are lowercased.
  3. Try and verify the signature

What should happen:

  1. The signature is verified correctly

What happens:

  1. The signature verification will fail

Logs

  • Logs no logs

Extra info:
@OGKevin OGKevin added this to the 0.13.1 milestone Nov 28, 2017
@OGKevin OGKevin self-assigned this Nov 28, 2017
This was referenced Nov 28, 2017
Closed
Closed
Closed
@OGKevin OGKevin modified the milestones: 0.13.1, 0.12.5 Nov 28, 2017
OGKevin added a commit that referenced this issue Dec 21, 2017
@OGKevin
Check headers casing. (#75)
bc5c37e
OGKevin added a commit that referenced this issue Dec 21, 2017
@OGKevin
Exctracted the case chech in a seperate method. (#75)
21fd016
@OGKevin OGKevin modified the milestones: 0.12.5, 0.12.4 Dec 21, 2017
This was referenced Dec 21, 2017
Merged
Open
OGKevin added a commit that referenced this issue Dec 21, 2017
@OGKevin
Removed mixed from doc block. (#75)
ace996f
@OGKevin OGKevin mentioned this issue Dec 24, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@OGKevin OGKevin

Labels
enhancement
Projects
None yet
Milestone
0.12.4
Development

No branches or pull requests
1 participant
@OGKevin