add new DPoPTokenSerializer class for [de]serializing DPoPToken
for guillp#122
@guillp one note here: this duplicates the existing code in AuthorizationRequestSerializer for [de]serializing DPoPKey. would you want a new DPoPKeySerializer class for that? I wasn't sure if you'd care more about the code duplication or the serializer class sprawl. I'm ok either way.
Codecov Report
:x: Patch coverage is 95.83333% with 1 line in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 99.94%. Comparing base (49f0739) to head (69e6107).
:warning: Report is 19 commits behind head on main.
| Files with missing lines | Patch % | Lines |
|---|---|---|
| requests_oauth2client/dpop.py | 95.45% | 1 Missing :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## main #174 +/- ##
===========================================
- Coverage 100.00% 99.94% -0.06%
===========================================
Files 20 20
Lines 1974 1995 +21
===========================================
+ Hits 1974 1994 +20
- Misses 0 1 +1
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
@guillp friendly ping? I'd love to get back to using requests_oauth2client from upstream here instead of my fork!...but I need this PR first.
Hi @snarfed . This is not forgotten. But I didn't like to create 1 serializer class per token type. So I just started to refactor and improves all serializers in #196.
closing this since #196 implements serialization of DPoPTokens