Project

Profile

Help

Send by e-mail

Issue #8330

RemoteSerializer does not include `headers` and field incorrectly marks as Required

Added by bmbouter about 1 month ago. Updated 30 days ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
mdellweg
Category:
-
Sprint/Milestone:
3.11.0
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:

  1. The RemoteSerializer does actually name headers to be an included field in the metaclass
  2. 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

Revision 78c55fcc View on GitHub
Added by mdellweg about 1 month ago

Fix headers field on the RemoteSerializer

fixes #8330 https://pulp.plan.io/issues/8330

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

#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

Send by e-mail

Also available in: Atom PDF