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

Ferry Huberts (spam-protected)
Sun Feb 21 18:56:13 CET 2016


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.

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(-)

-- 
2.5.0




More information about the Olsr-dev mailing list