acme.messages.OrderResource: Make roundtrippable through JSON #9617
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Right now if you to_json() an
OrderResource
and later deserialize it, theAuthorizationResource
objects don't come back through the round-trip (they just get de-jsonified as frozendicts and worse, they can't even be passed toAuthorizationResource.from_json
because frozendicts aren't dicts). In addition, thecsr_pem
field gets encoded as an array of integers, which definitely does not get de-jsonified into what we want.Fix these by adding an encoder to
authorizations
and encoder and decoder tocsr_pem
.Fixes #9615
Pull Request Checklist
master
section ofcertbot/CHANGELOG.md
to include a description of the change being made.AUTHORS.md
if you like.