Commit Graph

7 Commits

Author SHA1 Message Date
Craig Tiller
d6c98df792 clang-format all source 2015-08-18 09:33:44 -07:00
Eric Anderson
5b53e35ba4 Remove HTTP mappings from status.h
The mappings are a combination of useless, inaccurate, and harmful.
Leave them out until they are helpful and correct.

Fundamental issues:
 - If in need of mappings, a user would need HTTP -> gRPC code mappings.
   But the docs don't provide that since the mappings are not 1:1.
 - There _is no_ gRPC -> HTTP code mapping taking place in gRPC. This
   may change in the future, but until then the docs are misleading.

But even given those were resolved, there are additional issues. Two
obvious examples:
 - 501 Not Implemented is refering to HTTP methods such as GET and POST
   not being implement. That should not be used for gRPC methods not
   found (UNIMPLEMENTED).
 - 404 Not Found is for when the resource is not found. In gRPC, the
   resource is the gRPC method, not the parameters (NOT_FOUND). If there
   is a REST conversion layer on top of gRPC, then 404 would be
   appropriate, but that just shows that the mapping can not be
   canonical.
2015-03-02 09:42:02 -08:00
Nicolas "Pixel" Noble
1ff52d5278 Guard headers tool. 2015-03-01 06:18:14 +01:00
Craig Tiller
190d360def Add missing new-lines at end of file 2015-02-18 09:23:38 -08:00
Craig Tiller
0605995e55 Update copyright to 2015 2015-02-18 08:34:56 -08:00
Craig Tiller
b5dcec5a2e clang-format codebase 2015-01-13 11:13:42 -08:00
Nicolas Noble
b7ebd3b8c6 Initial import. 2014-11-26 16:33:03 -08:00