Project

Profile

Help

Issue #8330

closed

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

Added by bmbouter over 1 year ago. Updated over 1 year ago.

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

Updated by mdellweg over 1 year ago

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

Updated by pulpbot over 1 year ago

  • Status changed from ASSIGNED to POST

Added by mdellweg over 1 year ago

Revision 78c55fcc

Fix headers field on the RemoteSerializer

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

Actions #3

Updated by mdellweg over 1 year ago

  • Status changed from POST to MODIFIED
Actions #4

Updated by ipanova@redhat.com over 1 year ago

  • Sprint/Milestone set to 3.11.0
Actions #5

Updated by ipanova@redhat.com over 1 year ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF