)]}'
{"nova/cmd/manage.py":[{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"6b90aced25a36cdfdbbc54f58eb73e125a5a545e","unresolved":false,"context_lines":[{"line_number":1213,"context_line":"            is_parent \u003d True"},{"line_number":1214,"context_line":"        values \u003d {\u0027name\u0027: name,"},{"line_number":1215,"context_line":"                  \u0027is_parent\u0027: is_parent,"},{"line_number":1216,"context_line":"                  \u0027transport_url\u0027: urllib.unquote(str(transport_url)),"},{"line_number":1217,"context_line":"                  \u0027weight_offset\u0027: float(woffset),"},{"line_number":1218,"context_line":"                  \u0027weight_scale\u0027: float(wscale)}"},{"line_number":1219,"context_line":"        ctxt \u003d context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa963961_3904d513","line":1216,"updated":"2015-03-26 20:02:14.000000000","message":"What about handling the unquoting in _create_transport_hosts in case that gets used somewhere else later?","commit_id":"ec635b07e350e282a69a14d7328a83fc5b843989"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"2e08c6ca7ed95c925d9e7a748dbdd85da2e73030","unresolved":false,"context_lines":[{"line_number":1213,"context_line":"            is_parent \u003d True"},{"line_number":1214,"context_line":"        values \u003d {\u0027name\u0027: name,"},{"line_number":1215,"context_line":"                  \u0027is_parent\u0027: is_parent,"},{"line_number":1216,"context_line":"                  \u0027transport_url\u0027: urllib.unquote(str(transport_url)),"},{"line_number":1217,"context_line":"                  \u0027weight_offset\u0027: float(woffset),"},{"line_number":1218,"context_line":"                  \u0027weight_scale\u0027: float(wscale)}"},{"line_number":1219,"context_line":"        ctxt \u003d context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":2,"id":"da9b358b_80d69ec3","line":1216,"in_reply_to":"fa963961_2a091196","updated":"2015-03-30 19:22:23.000000000","message":"I\u0027m totally fine with this, just curious if you had looked into fixing it there.  The TransportHost objects are being created with encoded data which might be problematic if they were used for anything beyond getting a string here.","commit_id":"ec635b07e350e282a69a14d7328a83fc5b843989"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"7f24446ddd34ae67da9a95864f1e388760465a26","unresolved":false,"context_lines":[{"line_number":1213,"context_line":"            is_parent \u003d True"},{"line_number":1214,"context_line":"        values \u003d {\u0027name\u0027: name,"},{"line_number":1215,"context_line":"                  \u0027is_parent\u0027: is_parent,"},{"line_number":1216,"context_line":"                  \u0027transport_url\u0027: urllib.unquote(str(transport_url)),"},{"line_number":1217,"context_line":"                  \u0027weight_offset\u0027: float(woffset),"},{"line_number":1218,"context_line":"                  \u0027weight_scale\u0027: float(wscale)}"},{"line_number":1219,"context_line":"        ctxt \u003d context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa963961_2a091196","line":1216,"in_reply_to":"fa963961_3904d513","updated":"2015-03-26 21:49:23.000000000","message":"Andrew, _create_transport_hosts returns an list of messaging.TransportHost objects. we have this problem once we turn it into a string. So this focused fix is the best approach i feel.","commit_id":"ec635b07e350e282a69a14d7328a83fc5b843989"}]}
