[PATCHv2 02/10] REST: Add base configuration hooks for a REST API

Finucane, Stephen stephen.finucane at intel.com
Thu May 19 07:58:45 AEST 2016


On 17 May 13:17, Finucane, Stephen wrote:
> On 10 May 17:39, Andy Doan wrote:
> > This adds the ability to expose a REST API based on the Django REST
> > framework project. Since this project isn't packaged in most current
> > distributions, we ensure that its both installed and enabled before
> > trying to use it.
> > 
> > Signed-off-by: Andy Doan <andy.doan at linaro.org>
> > Inspired-by: Damien Lespiau <damien.lespiau at intel.com>
> 
> Reviewed-by: Stephen Finucane <stephen.finucane at intel.com>

Actually, I do have one comment after playing around with this a little
more: what's the possibility of using a 'Link' header instead of
a payload envelope? Seems to be pretty easy to do [1], it's a proposed
RFC standard [2], and it's consistent with the GitHub API (which I
happen to admire) [3]. Certainly won't block on it though. Thoughts?

Stephen

[1] http://www.django-rest-framework.org/api-guide/pagination/#header-based-pagination
[2] https://tools.ietf.org/html/rfc5988#section-5
[3] https://developer.github.com/guides/traversing-with-pagination/


More information about the Patchwork mailing list