Project

Profile

Help

Issue #5299

closed

Bindings for pulp \ pulp ansible return python `repr()` for json fields

Added by osapryki over 4 years ago. Updated over 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Assignee:
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
3. High
Platform Release:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 57
Quarter:

Description

Fields described in DRF serializers as `JSONField()` [1] are included in swagger schema as `string` [2] fields, which leads to invalid deserialization on a client side when using pulp bindings.

Those fields deserialized as a `str` type containing repr() result of json field passed from server.

This issue is a blocker, because it cannot be reliably workarounded on a client side as there is no reliable way to deserialize python repr() back to a normal object.

[1] https://github.com/pulp/pulp_ansible/blob/master/pulp_ansible/app/serializers.py#L197
[2] https://pulp-ansible.readthedocs.io/en/latest/restapi.html#operation/content_ansible_collections_list

Actions #1

Updated by daviddavis over 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee changed from bmbouter to fao89
  • Sprint set to Sprint 57

Added by Fabricio Aguiar over 4 years ago

Revision 28f40e6b | View on GitHub

change JSONField to DictField

closes #5299 https://pulp.plan.io/issues/5299

Actions #2

Updated by Anonymous over 4 years ago

  • Status changed from ASSIGNED to MODIFIED
Actions #4

Updated by bmbouter over 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF