Code review comment for lp:~gmb/maas/fix-null-when-releasing-ip-bug-1383668

Revision history for this message
Graham Binns (gmb) wrote :

> Oh I hadn't realised that text was getting sent, I presumed it was a json
> artifact. Let's make it stop sending those back, they're useless.

Well, they're a JSON artefact insofar as when the API method returns None it gets converted to u'null', and that becomes the response body.

I can't think of a situation (right now) when we'd want to return None deliberately, but I'm leery of getting rid of it, just-in-case.

« Back to merge proposal