Skip to content

Conversation

@blag
Copy link
Contributor

@blag blag commented Sep 7, 2020

Closes #71.
Resolves #72.
Resolves #74.
Closes #93 (supercedes).
Resolves #126.

This PR makes key handling more robust by properly handling keys that are dictionaries and (more importantly) keys that are not dictionaries.

@blag blag added the bug label Sep 7, 2020
@codecov
Copy link

codecov bot commented Sep 7, 2020

Codecov Report

Merging #187 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #187   +/-   ##
=======================================
  Coverage   93.49%   93.49%           
=======================================
  Files          16       16           
  Lines        1723     1723           
=======================================
  Hits         1611     1611           
  Misses        112      112           
Impacted Files Coverage Δ
jose/jws.py 96.72% <100.00%> (ø)

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 99ec142...46302d7. Read the comment docs.

@blag blag merged commit 2161958 into mpdavis:master Sep 7, 2020
@blag blag deleted the allow-bytes-secret branch September 7, 2020 23:27
@dp-rufus
Copy link

dp-rufus commented Nov 6, 2020

Thanks for resolving this! Please could you push a new release to PyPI - the current version is missing this fix.

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.

If key is bytes, encode works, decode fails base64url_decode TypeError: must be str, not bytes jws.verify type error

3 participants