[Olsr-dev] [PATCH v1 00/10] txtinfo & jsoninfo: HTTP error status codes

[Ferry Huberts]

On 21/02/16 18:56, Ferry Huberts wrote:
> From: Ferry Huberts <(spam-protected)>
>
> This series makes jsoninfo and txtinfo send HTTP error status codes
> on invalid commands and empty output.
>
> This is a MAJOR change.


No comments???

If nobody has comments this week, I'll commit next Monday.


>
> The series starts with a few fixes, then ensures that empty requests and those
> for the root document (/) are interpreted as '/all' to ensure (some) backward
> compatibility.
>
> Subsequently it works toward returning HTTP error status codes on relevant
> conditions.
>
>
> Please revew.
>
>
> Ferry Huberts (10):
>    info: requests that are too large must set s to < 0
>    info: also handle empty requests
>    info: rename an HTTP status define
>    info: requesting / now is equivalent to /all
>    info: prepare for more HTTP status codes
>    info: add function to the function table to output an error
>    info: respond with "not found" (404) on invalid commands
>    info: respond with "no content" (204) on empty output
>    txtinfo: /config & /plugins are not implemented: return "not found"
>      (404)
>    changelog: update for info/jsoninfo/txtinfo changes
>
>   CHANGELOG                         |  24 +++++--
>   lib/info/http_headers.c           |   5 +-
>   lib/info/http_headers.h           |  22 ++++++-
>   lib/info/info_types.h             |   2 +
>   lib/info/olsrd_info.c             | 134 ++++++++++++++++++++++++--------------
>   lib/jsoninfo/src/olsrd_jsoninfo.c |  28 ++++++++
>   lib/jsoninfo/src/olsrd_jsoninfo.h |   1 +
>   lib/jsoninfo/src/olsrd_plugin.c   |   1 +
>   lib/txtinfo/src/olsrd_plugin.c    |   1 +
>   lib/txtinfo/src/olsrd_txtinfo.c   |  26 +++++++-
>   lib/txtinfo/src/olsrd_txtinfo.h   |   1 +
>   11 files changed, 181 insertions(+), 64 deletions(-)
>

-- 
Ferry Huberts