[Pulp-dev] breaking change: accept Null instead of empty string

Brian Bouterse bbouters at redhat.com
Tue May 21 17:09:28 UTC 2019


On Tue, May 21, 2019 at 12:25 PM Dennis Kliban <dkliban at redhat.com> wrote:

> Pulp 3's REST API does not accept Null as a value for a string field. The
> only way for a user to unset a string field is to pass in an empty string.
> This makes it awkward to integrate Pulp with other systems that use Null to
> represent unset fields. We discovered this usability problem when
> investigating issue with the bindings[0].
>
> I propose that we switch all fields in pulpcore's REST API to accept null
> instead of empty strings and raise a validation error when an empty string
> is passed.
>
> Does this change make sense? Does anyone have objections?
>
This makes sense to me. +1 to adopting this in core and all plugins.


> [0] https://pulp.plan.io/issues/4676
>
>
>  - Dennis
> _______________________________________________
> Pulp-dev mailing list
> Pulp-dev at redhat.com
> https://www.redhat.com/mailman/listinfo/pulp-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/pulp-dev/attachments/20190521/37bfb1eb/attachment.htm>


More information about the Pulp-dev mailing list