Project

Profile

Help

Send by e-mail

Issue #8330

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

Added by bmbouter 5 months ago. Updated 2 months 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 5 months ago

Fix headers field on the RemoteSerializer

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

History

#1 Updated by mdellweg 5 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to mdellweg

#2 Updated by pulpbot 5 months ago

  • Status changed from ASSIGNED to POST

#3 Updated by mdellweg 5 months ago

  • Status changed from POST to MODIFIED

#4 Updated by ipanova@redhat.com 5 months ago

  • Sprint/Milestone set to 3.11.0

#5 Updated by ipanova@redhat.com 5 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Send by e-mail

Also available in: Atom PDF