)]}'
{"gnocchi/indexer/__init__.py":[{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"31b3877a1fc64a3ed047f59c75b7292ab96bd822","unresolved":false,"context_lines":[{"line_number":113,"context_line":"    def list_resources(resource_type\u003d\u0027generic\u0027, started_after\u003dNone,"},{"line_number":114,"context_line":"                       ended_before\u003dNone,"},{"line_number":115,"context_line":"                       attributes_filter\u003dNone,"},{"line_number":116,"context_line":"                       details\u003dFalse):"},{"line_number":117,"context_line":"        raise NotImplementedError"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    @staticmethod"}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_18517330","line":116,"updated":"2014-07-23 13:28:38.000000000","message":"Here\u0027s a wacky idea ...\n\nInstead of exposing the details field as an explicit query param in the URI path, how about using an accept-extension[1] instead specified in the headers?\n\nHere\u0027s an example of this pattern [2].\n\nSo the query path could remain focused on selecting the target resources, while the Accept header describes the format required, e.g.:\n\n  GET /v2/resources?started_after\u003dyesterday\n  Accept: application/json ; details \u003d verbose\n\nDunno how convenient this would be to implement with pecan, just a thought ...\n\n* [1] http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html\n* [2] https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Virtualization/3.0/html-single/REST_API_Guide/#sect-REST_API_Guide-Common_Features-Collections-Listing_Resource_Subcollections","commit_id":"c694378878dd50070720cf1e7ee1e897feae9f81"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"36b8179a3c6975b355704e987e24a2ca09589b47","unresolved":false,"context_lines":[{"line_number":113,"context_line":"    def list_resources(resource_type\u003d\u0027generic\u0027, started_after\u003dNone,"},{"line_number":114,"context_line":"                       ended_before\u003dNone,"},{"line_number":115,"context_line":"                       attributes_filter\u003dNone,"},{"line_number":116,"context_line":"                       details\u003dFalse):"},{"line_number":117,"context_line":"        raise NotImplementedError"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    @staticmethod"}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_0e673feb","line":116,"in_reply_to":"baada198_18517330","updated":"2014-07-24 14:43:19.000000000","message":"Sounds like a good idea; I\u0027ll do that in the patch exporting that feature in the REST API.","commit_id":"c694378878dd50070720cf1e7ee1e897feae9f81"}]}
