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 #43

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

Make sure received signatures headers are correctly cased #43

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

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:

Reference
@OGKevin OGKevin added this to the 0.12.5 milestone Nov 28, 2017
@OGKevin OGKevin self-assigned this Nov 28, 2017
@OGKevin OGKevin modified the milestones: 0.12.5, 0.12.4 Dec 21, 2017
OGKevin added a commit that referenced this issue Dec 21, 2017
@OGKevin
Added method to ensure that reponse headers are correclty cased befor…
e1c7f8e 
…e verification. (#43)
@OGKevin OGKevin mentioned this issue Dec 21, 2017
Merged
OGKevin added a commit that referenced this issue Dec 21, 2017
@OGKevin
Removed wild imports. (#43)
9a1c2ef
OGKevin added a commit that referenced this issue Dec 21, 2017
@OGKevin
Renamed ensureHeaderCorrectlyCased to getHeaderNameCorrectlyCased. (#43)
0d91203
OGKevin added a commit that referenced this issue Dec 21, 2017
@OGKevin
Renamed substring constants. (#43)
cbf66ac
OGKevin added a commit that referenced this issue Dec 21, 2017
@OGKevin
End index of first char instead of last char. (#43)
f9e252b
@OGKevin OGKevin mentioned this issue Dec 21, 2017
Open
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