Skip to content

OrderedDict for headers#81

Closed
Fufutako wants to merge 5 commits intompdavis:masterfrom
Fufutako:predictible_jwtheaders
Closed

OrderedDict for headers#81
Fufutako wants to merge 5 commits intompdavis:masterfrom
Fufutako:predictible_jwtheaders

Conversation

@Fufutako
Copy link

@Fufutako Fufutako commented Feb 9, 2018

Fixes #80
Construct headers from an ordered dictionary instead a standard dict.
This makes it possible not to send the headers in the token and rebuild
them on the server side when it is used by client. Indeed, for this the
json must be predictable. What do not allow std python dictionaries.

Mickael Viey added 5 commits February 9, 2018 13:54
Construct headers from an ordered dictionary instead a standard dict.
This makes it possible not to send the headers in the token and rebuild
them on the server side when it is used by client. Indeed, for this the
json must be predictable. What do not allow std python dictionaries.
@codecov-io
Copy link

Codecov Report

Merging #81 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #81   +/-   ##
=======================================
  Coverage   93.69%   93.69%           
=======================================
  Files          12       12           
  Lines         841      841           
=======================================
  Hits          788      788           
  Misses         53       53
Impacted Files Coverage Δ
jose/jwt.py 94.19% <100%> (ø) ⬆️
jose/jws.py 94.11% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 28cc671...cb784b8. Read the comment docs.

@threaz
Copy link

threaz commented Nov 29, 2018

Hi. What's the current status of this MR? Can it be merged?

@zejn
Copy link
Collaborator

zejn commented Apr 10, 2019

Implemented this via #136.

@zejn zejn closed this Apr 10, 2019
Sign up for free to 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.

4 participants