)]}'
{"tests/test_titles.py":[{"author":{"_account_id":6804,"name":"bruce-benjamin","email":"bruce.benjamin@jhuapl.edu","username":"bruce-benjamin"},"change_message_id":"3ad873b843c7f6b8ed87b64e4c6e536f3e0bbcb5","unresolved":false,"context_lines":[{"line_number":74,"context_line":"                continue"},{"line_number":75,"context_line":"            # Allow lines which do not contain any whitespace"},{"line_number":76,"context_line":"            if re.match(\"\\s*[^\\s]+$\", line):"},{"line_number":77,"context_line":"                continue"},{"line_number":78,"context_line":"            self.assertTrue("},{"line_number":79,"context_line":"                len(line) \u003c 80,"},{"line_number":80,"context_line":"                msg\u003d\"%s:%d: Line limited to a maximum of 79 characters.\" %"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa1b9901_484ebd97","line":77,"updated":"2015-08-21 22:40:16.000000000","message":"Could you add a comment that explains how this URL size will be limited to a reasonable value which shouldn\u0027t cause denial of service issues?  Thanks.","commit_id":"6469ac9258ed99b14c8d4473168a108aecd0061c"},{"author":{"_account_id":6804,"name":"bruce-benjamin","email":"bruce.benjamin@jhuapl.edu","username":"bruce-benjamin"},"change_message_id":"85b28d0ac5fb143bcd7e26ac0cdf46d57243b094","unresolved":false,"context_lines":[{"line_number":74,"context_line":"                continue"},{"line_number":75,"context_line":"            # Allow lines which do not contain any whitespace"},{"line_number":76,"context_line":"            if re.match(\"\\s*[^\\s]+$\", line):"},{"line_number":77,"context_line":"                continue"},{"line_number":78,"context_line":"            self.assertTrue("},{"line_number":79,"context_line":"                len(line) \u003c 80,"},{"line_number":80,"context_line":"                msg\u003d\"%s:%d: Line limited to a maximum of 79 characters.\" %"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa1b9901_93bfe8fb","line":77,"in_reply_to":"fa1b9901_393a99ce","updated":"2015-08-24 17:03:33.000000000","message":"I see - for solely testing this looks like it would be ok without any further explanation.  Thanks.","commit_id":"6469ac9258ed99b14c8d4473168a108aecd0061c"},{"author":{"_account_id":8688,"name":"Alexis Lee","email":"openstack@lxsli.co.uk","username":"lxsli"},"change_message_id":"99c43dc799b5686e0b4591bbac70f00d8d52cb21","unresolved":false,"context_lines":[{"line_number":74,"context_line":"                continue"},{"line_number":75,"context_line":"            # Allow lines which do not contain any whitespace"},{"line_number":76,"context_line":"            if re.match(\"\\s*[^\\s]+$\", line):"},{"line_number":77,"context_line":"                continue"},{"line_number":78,"context_line":"            self.assertTrue("},{"line_number":79,"context_line":"                len(line) \u003c 80,"},{"line_number":80,"context_line":"                msg\u003d\"%s:%d: Line limited to a maximum of 79 characters.\" %"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa1b9901_393a99ce","line":77,"in_reply_to":"fa1b9901_484ebd97","updated":"2015-08-24 08:21:39.000000000","message":"Sorry, what are you concerned about being DOS\u0027d? Gerrit? Remember this is just a test, any patch would still have to pass manual code review before being merged.\n\nI have no objection to limiting the URL to say 256 characters beyond the additional code complexity required. I don\u0027t see the necessity though.","commit_id":"6469ac9258ed99b14c8d4473168a108aecd0061c"}]}