Issue #8330
RemoteSerializer does not include `headers` and field incorrectly marks as Required
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:
Description
In 3.10 a new headers
field was added for all Remotes with this PR https://github.com/pulp/pulpcore/pull/1041/files
There are two issues with this:
- The
RemoteSerializer
does actually nameheaders
to be an included field in the metaclass - The field defined does not allow it to be optional, which it needs to be optional.
Because of (1) problem (2) was hidden during 3.10 usage.
Associated revisions
History
#1
Updated by mdellweg about 1 month ago
- Status changed from NEW to ASSIGNED
- Assignee set to mdellweg
#2
Updated by pulpbot about 1 month ago
- Status changed from ASSIGNED to POST
#3
Updated by bhenmia0 about 1 month ago
Thanks for the quick reply [[https://get-mobdro.com]] Regards, [[https://get-tubemate.com]]
#4
Updated by mdellweg about 1 month ago
- Status changed from POST to MODIFIED
Applied in changeset pulpcore|78c55fccefbceef7a20b3bd16f917b32e1125935.
#5
Updated by ipanova@redhat.com about 1 month ago
- Sprint/Milestone set to 3.11.0
#6
Updated by ipanova@redhat.com 30 days ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Please register to edit this issue
Fix headers field on the RemoteSerializer
fixes #8330 https://pulp.plan.io/issues/8330