Project

Profile

Help

Actions
Send by e-mail Copy link

Issue #8330

closed

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

Added by bmbouter over 3 years ago. Updated over 3 years 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.

Actions
Copy link
 #1

Updated by mdellweg over 3 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to mdellweg
Actions
Copy link
 #2

Updated by pulpbot over 3 years ago

  • Status changed from ASSIGNED to POST

Added by mdellweg over 3 years ago

Revision 78c55fcc | View on GitHub

Fix headers field on the RemoteSerializer

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

Actions
Copy link
 #3

Updated by mdellweg over 3 years ago

  • Status changed from POST to MODIFIED
Actions
Copy link
 #4

Updated by ipanova@redhat.com over 3 years ago

  • Sprint/Milestone set to 3.11.0
Actions
Copy link
 #5

Updated by ipanova@redhat.com over 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
Send by e-mail Copy link

Also available in: Atom PDF