)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"ab46fa338d40e28d5a4a5218e18bb535b7122b32","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ken\u0027ichi Ohmichi \u003cken-oomichi@wx.jp.nec.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-10-14 15:50:07 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add http400 for nonexistent resource"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ie6e461a133a62de1c87b4b6d963c02fa91aca63c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"5a2ca52d_4fd2e7d3","line":7,"updated":"2015-10-15 11:00:18.000000000","message":"I\u0027d like to see slightly more in the commit message otherwise someone reviewing the commit history will get the wrong idea about what is happening here.","commit_id":"250d48820f2a7a5f0d2430b305b23f94fbdcab08"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"fcace42d2b08dfe9fcb4d6c459de5d649534cde2","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ken\u0027ichi Ohmichi \u003cken-oomichi@wx.jp.nec.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-10-14 15:50:07 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add http400 for nonexistent resource"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ie6e461a133a62de1c87b4b6d963c02fa91aca63c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"5a2ca52d_58594ffd","line":7,"in_reply_to":"5a2ca52d_4fd2e7d3","updated":"2015-10-15 11:42:58.000000000","message":"Chris, thanks for your review.\n\nOK, I will try it :)","commit_id":"250d48820f2a7a5f0d2430b305b23f94fbdcab08"}],"guidelines/http.rst":[{"author":{"_account_id":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},"change_message_id":"12711661697298cf17b015fa38b4d464c24a6659","unresolved":false,"context_lines":[{"line_number":154,"context_line":"    quota -- for a subsequent request to succeed when quotas are exceeded the"},{"line_number":155,"context_line":"    server environment must change."},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"* If a request contains nonexistent resource on the body(not url), the code"},{"line_number":158,"context_line":"  should be **400 Bad Request**. Do **not** use **404 NotFound** because"},{"line_number":159,"context_line":"  :rfc: `http://tools.ietf.org/html/rfc2616.html#section-10.4.5` mentions"},{"line_number":160,"context_line":"  **The server has not found anything matching the Request-URI** for 404."}],"source_content_type":"text/x-rst","patch_set":3,"id":"da20952f_7d3cb82e","line":157,"updated":"2015-09-08 06:47:22.000000000","message":"I prefer like `If a request contains nonexistent resource which are associated to target resource (target uri),`","commit_id":"88e44fdc9fa5fecc928c167e11cf10ad13df9d42"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"2db90ac04d45c00507a2f0e2fdd7a8012eb0184e","unresolved":false,"context_lines":[{"line_number":116,"context_line":""},{"line_number":117,"context_line":"* For all other successful requests, the return code should be **200 OK**."},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"* If a request attempts to put a resource into a state which it is"},{"line_number":120,"context_line":"  already in (eg locking an instance which is already locked), the return code"},{"line_number":121,"context_line":"  should be in the **2xx Successful** range (usually matching the return code"},{"line_number":122,"context_line":"  which would be given if the state had changed). It is not appropriate to use"}],"source_content_type":"text/x-rst","patch_set":9,"id":"da20952f_242b4435","line":119,"updated":"2015-09-09 14:37:22.000000000","message":"here is eg.","commit_id":"16dba6933699362ed6bf754ae9097a11a7af113d"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"67cc182861b157e8fb0d6edafb494e1b8cd4cb3e","unresolved":false,"context_lines":[{"line_number":154,"context_line":"    quota -- for a subsequent request to succeed when quotas are exceeded the"},{"line_number":155,"context_line":"    server environment must change."},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"* If a request contains a reference of nonexistent resource in the body"},{"line_number":158,"context_line":"  (not url), the code should be **400 Bad Request**. Do **not** use **404"},{"line_number":159,"context_line":"  NotFound** because :rfc:`2616` mentions **The server has not found anything"},{"line_number":160,"context_line":"  matching the Request-URI** for 404. (eg: the specified flavor doesn\u0027t exist"}],"source_content_type":"text/x-rst","patch_set":9,"id":"da20952f_61da8e19","line":157,"updated":"2015-09-09 14:21:35.000000000","message":"i think this would be clearer as \"If a request contains a reference to a nonexistent resource...\"","commit_id":"16dba6933699362ed6bf754ae9097a11a7af113d"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"2db90ac04d45c00507a2f0e2fdd7a8012eb0184e","unresolved":false,"context_lines":[{"line_number":154,"context_line":"    quota -- for a subsequent request to succeed when quotas are exceeded the"},{"line_number":155,"context_line":"    server environment must change."},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"* If a request contains a reference of nonexistent resource in the body"},{"line_number":158,"context_line":"  (not url), the code should be **400 Bad Request**. Do **not** use **404"},{"line_number":159,"context_line":"  NotFound** because :rfc:`2616` mentions **The server has not found anything"},{"line_number":160,"context_line":"  matching the Request-URI** for 404. (eg: the specified flavor doesn\u0027t exist"}],"source_content_type":"text/x-rst","patch_set":9,"id":"da20952f_24cfc4b4","line":157,"in_reply_to":"da20952f_61da8e19","updated":"2015-09-09 14:37:22.000000000","message":"Thanks, will do it soon. :)","commit_id":"16dba6933699362ed6bf754ae9097a11a7af113d"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"67cc182861b157e8fb0d6edafb494e1b8cd4cb3e","unresolved":false,"context_lines":[{"line_number":157,"context_line":"* If a request contains a reference of nonexistent resource in the body"},{"line_number":158,"context_line":"  (not url), the code should be **400 Bad Request**. Do **not** use **404"},{"line_number":159,"context_line":"  NotFound** because :rfc:`2616` mentions **The server has not found anything"},{"line_number":160,"context_line":"  matching the Request-URI** for 404. (eg: the specified flavor doesn\u0027t exist"},{"line_number":161,"context_line":"  when creating a virtual machine, the code should be 400)"},{"line_number":162,"context_line":""},{"line_number":163,"context_line":"* If a request is made to a known resource URI, but the HTTP method used for"}],"source_content_type":"text/x-rst","patch_set":9,"id":"da20952f_61f12e95","line":160,"updated":"2015-09-09 14:21:35.000000000","message":"instead of \"eg:\" we should use \"for example,\" as per the documentation conventions","commit_id":"16dba6933699362ed6bf754ae9097a11a7af113d"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"2db90ac04d45c00507a2f0e2fdd7a8012eb0184e","unresolved":false,"context_lines":[{"line_number":157,"context_line":"* If a request contains a reference of nonexistent resource in the body"},{"line_number":158,"context_line":"  (not url), the code should be **400 Bad Request**. Do **not** use **404"},{"line_number":159,"context_line":"  NotFound** because :rfc:`2616` mentions **The server has not found anything"},{"line_number":160,"context_line":"  matching the Request-URI** for 404. (eg: the specified flavor doesn\u0027t exist"},{"line_number":161,"context_line":"  when creating a virtual machine, the code should be 400)"},{"line_number":162,"context_line":""},{"line_number":163,"context_line":"* If a request is made to a known resource URI, but the HTTP method used for"}],"source_content_type":"text/x-rst","patch_set":9,"id":"da20952f_2487c413","line":160,"in_reply_to":"da20952f_61f12e95","updated":"2015-09-09 14:37:22.000000000","message":"ok, will do it. but there are some inconsistencies about that.\nbut that should be out of scope in this patch.","commit_id":"16dba6933699362ed6bf754ae9097a11a7af113d"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"2db90ac04d45c00507a2f0e2fdd7a8012eb0184e","unresolved":false,"context_lines":[{"line_number":206,"context_line":"defining characteristic of POST isn\u0027t that it creates items, but that"},{"line_number":207,"context_line":"you POST to a URI that\u0027s different than the resource you get"},{"line_number":208,"context_line":"back. POST is therefor also appropriate for bulk operations like"},{"line_number":209,"context_line":"multiple update, or triggering some arbitrary other actions beyond"},{"line_number":210,"context_line":"resource creation (i.e. reboot a server)."},{"line_number":211,"context_line":""},{"line_number":212,"context_line":"**TODO**: HEAD is weird in a bunch of our wsgi frameworks and you"}],"source_content_type":"text/x-rst","patch_set":9,"id":"da20952f_e43f8c85","line":209,"updated":"2015-09-09 14:37:22.000000000","message":"here is i.e.","commit_id":"16dba6933699362ed6bf754ae9097a11a7af113d"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"54042067952e66cfa62a9ea697a1e40806f1f08a","unresolved":false,"context_lines":[{"line_number":155,"context_line":"    server environment must change."},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"* If a request contains a reference to a nonexistent resource in the body"},{"line_number":158,"context_line":"  (not url), the code should be **400 Bad Request**. Do **not** use **404"},{"line_number":159,"context_line":"  NotFound** because :rfc:`2616` mentions **The server has not found anything"},{"line_number":160,"context_line":"  matching the Request-URI** for 404. (For example, the specified flavor"},{"line_number":161,"context_line":"  doesn\u0027t exist when creating a virtual machine, the code should be 400)"}],"source_content_type":"text/x-rst","patch_set":10,"id":"7a2fa921_ee1fdcb4","line":158,"updated":"2015-10-09 09:27:32.000000000","message":"From same RFC:\n\n10.4.1 400 Bad Request\n\n   The request could not be understood by the server due to malformed\n   syntax. The client SHOULD NOT repeat the request without\n   modifications.\n\nNon-existent ID is not a malformed syntax AFAIUI.","commit_id":"07d9b0656cb437142ab49f419aec5e7a0980cc5a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"f127b2244daf843c9206bc667d0e0fe13a18cc20","unresolved":false,"context_lines":[{"line_number":155,"context_line":"    server environment must change."},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"* If a request contains a reference to a nonexistent resource in the body"},{"line_number":158,"context_line":"  (not url), the code should be **400 Bad Request**. Do **not** use **404"},{"line_number":159,"context_line":"  NotFound** because :rfc:`2616` mentions **The server has not found anything"},{"line_number":160,"context_line":"  matching the Request-URI** for 404. (For example, the specified flavor"},{"line_number":161,"context_line":"  doesn\u0027t exist when creating a virtual machine, the code should be 400)"}],"source_content_type":"text/x-rst","patch_set":10,"id":"7a2fa921_89566204","line":158,"in_reply_to":"7a2fa921_ee1fdcb4","updated":"2015-10-09 09:33:35.000000000","message":"An alternative code for this situation may be 422 [1], but to be honest I think that 404 is simpler and more intuitive.\n\n[1] http://greenbytes.de/tech/webdav/rfc4918.html#STATUS_422","commit_id":"07d9b0656cb437142ab49f419aec5e7a0980cc5a"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"44806f35885016c62623e11c2cc1cb5d33d8161b","unresolved":false,"context_lines":[{"line_number":159,"context_line":"  NotFound** because :rfc:`7231#section-6.5.4` (section 6.5.4) mentions"},{"line_number":160,"context_line":"  **the origin server did not find a current representation for the target"},{"line_number":161,"context_line":"  resource** for 404 and **representation for the target resource** means"},{"line_number":162,"context_line":"  a URI. **422 Unprocessable Entity** also was an option for this situation"},{"line_number":163,"context_line":"  but do **not** use 422 because the code is not defined in :rfc:`7231` and"},{"line_number":164,"context_line":"  not standard. (For example, the specified flavor doesn\u0027t exist when creating"},{"line_number":165,"context_line":"  a virtual machine, the code should be 400)"}],"source_content_type":"text/x-rst","patch_set":12,"id":"5a2ca52d_7c59c859","line":162,"updated":"2015-10-14 15:38:42.000000000","message":"i think this should use present tense, so: \"**422 Unprocessable Entity** is also an option...\"","commit_id":"549debb168eecb0237cdb8cf5c8784487109867c"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"47ab111c4345d09399653f29db72678687769c1b","unresolved":false,"context_lines":[{"line_number":159,"context_line":"  NotFound** because :rfc:`7231#section-6.5.4` (section 6.5.4) mentions"},{"line_number":160,"context_line":"  **the origin server did not find a current representation for the target"},{"line_number":161,"context_line":"  resource** for 404 and **representation for the target resource** means"},{"line_number":162,"context_line":"  a URI. **422 Unprocessable Entity** also was an option for this situation"},{"line_number":163,"context_line":"  but do **not** use 422 because the code is not defined in :rfc:`7231` and"},{"line_number":164,"context_line":"  not standard. (For example, the specified flavor doesn\u0027t exist when creating"},{"line_number":165,"context_line":"  a virtual machine, the code should be 400)"}],"source_content_type":"text/x-rst","patch_set":12,"id":"5a2ca52d_7fa0da9f","line":162,"in_reply_to":"5a2ca52d_7c59c859","updated":"2015-10-14 15:48:48.000000000","message":"Tanks Michael,\n\nI will fix here :)","commit_id":"549debb168eecb0237cdb8cf5c8784487109867c"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"dac9f74bb671c07ecba09096be6176a59f761ac6","unresolved":false,"context_lines":[{"line_number":162,"context_line":"  a URI. **422 Unprocessable Entity** is also an option for this situation"},{"line_number":163,"context_line":"  but do **not** use 422 because the code is not defined in :rfc:`7231` and"},{"line_number":164,"context_line":"  not standard. (For example, the specified flavor doesn\u0027t exist when creating"},{"line_number":165,"context_line":"  a virtual machine, the code should be 400)"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"* If a request is made to a known resource URI, but the HTTP method used for"},{"line_number":168,"context_line":"  the request is not supported for that resource, the return code should be"}],"source_content_type":"text/x-rst","patch_set":14,"id":"3a29b11f_1a9ef6b8","line":165,"updated":"2015-10-22 22:23:21.000000000","message":"I tend to agree with the use of 400 here.\n\nI had to read the paragraph several times because it was not clear to me if by non-existent resource we referred also to something like /v2/some_random_stuff  - maybe clarify for dumb reader like me that the \"request body\" should contain a reference to a non-existing resource.","commit_id":"c973998ad65b7a5dbdce4cd645b2b07f6863ebc2"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"c8fba61bcc57212cef3f7b70ed78a87e212c9fae","unresolved":false,"context_lines":[{"line_number":162,"context_line":"  a URI. **422 Unprocessable Entity** is also an option for this situation"},{"line_number":163,"context_line":"  but do **not** use 422 because the code is not defined in :rfc:`7231` and"},{"line_number":164,"context_line":"  not standard. (For example, the specified flavor doesn\u0027t exist when creating"},{"line_number":165,"context_line":"  a virtual machine, the code should be 400)"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"* If a request is made to a known resource URI, but the HTTP method used for"},{"line_number":168,"context_line":"  the request is not supported for that resource, the return code should be"}],"source_content_type":"text/x-rst","patch_set":14,"id":"fa80f949_dd7ef3ad","line":165,"in_reply_to":"3a29b11f_1a9ef6b8","updated":"2015-11-04 01:25:09.000000000","message":"Hi Salvatore,\n\nI guess the above\n\n If a request contains a reference to a nonexistent resource in the body (not URI)\n\nis saying the same thing. Isn\u0027t that enough?\n\nI am not native English speaker, and sorry if something is unclear.","commit_id":"c973998ad65b7a5dbdce4cd645b2b07f6863ebc2"},{"author":{"_account_id":1112,"name":"Everett Toews","email":"everett.toews@rackspace.com","username":"everett-toews"},"change_message_id":"e1c91450ccca77bb78fe2592f680adebdfa4da4f","unresolved":false,"context_lines":[{"line_number":162,"context_line":"  a URI. **422 Unprocessable Entity** is also an option for this situation"},{"line_number":163,"context_line":"  but do **not** use 422 because the code is not defined in :rfc:`7231` and"},{"line_number":164,"context_line":"  not standard. (For example, the specified flavor doesn\u0027t exist when creating"},{"line_number":165,"context_line":"  a virtual machine, the code should be 400)"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"* If a request is made to a known resource URI, but the HTTP method used for"},{"line_number":168,"context_line":"  the request is not supported for that resource, the return code should be"}],"source_content_type":"text/x-rst","patch_set":14,"id":"fa80f949_6825c3a9","line":165,"in_reply_to":"fa80f949_dd7ef3ad","updated":"2015-11-04 01:36:32.000000000","message":"That\u0027s perfectly clear Ken\u0027ichi.","commit_id":"c973998ad65b7a5dbdce4cd645b2b07f6863ebc2"}]}