)]}'
{"openstack/load_balancer/v2/load_balancer.py":[{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"5eb8129bd787afa6147fe2997a0e68d8e1fa6a44","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        request.headers.update(headers)"},{"line_number":78,"context_line":"        if (hasattr(self, \u0027cascade\u0027) and isinstance(self.cascade, bool)"},{"line_number":79,"context_line":"                and self.cascade):"},{"line_number":80,"context_line":"            request.url +\u003d \u0027?cascade\u003d%s\u0027 % self.cascade"},{"line_number":81,"context_line":"        response \u003d session.delete(request.url,"},{"line_number":82,"context_line":"                                  headers\u003dheaders)"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"bf659307_acd93b23","line":80,"updated":"2018-03-23 18:26:32.000000000","message":"Instead of +\u003d \u0027?cascase\u0027 ... this should just pass params\u003d{\u0027cascade\u0027: True} as an argument to session.delete (keystoneauth/requests) will handle it for us.\n\nSo you could do:\n\n  params \u003d {}\n  if (hasattr(self, \u0027cascade\u0027) and isinstance(self.cascade, bool)\n          and self.cascade):\n    params[\u0027cascade\u0027] \u003d True\n\n  response \u003d session.delete(request.url,\n                            headers\u003dheaders,\n                            params\u003dparams)\n\nI think it\u0027ll be more resilient in the long term.","commit_id":"c112f92bb28c157757cfe5f4b3c3db3b55235d31"},{"author":{"_account_id":2245,"name":"Jacky Hu","email":"hudayou@hotmail.com","username":"hudayou"},"change_message_id":"1ef7f23260a1c5639f27442b8c56e3abc5d9d366","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        request.headers.update(headers)"},{"line_number":78,"context_line":"        if (hasattr(self, \u0027cascade\u0027) and isinstance(self.cascade, bool)"},{"line_number":79,"context_line":"                and self.cascade):"},{"line_number":80,"context_line":"            request.url +\u003d \u0027?cascade\u003d%s\u0027 % self.cascade"},{"line_number":81,"context_line":"        response \u003d session.delete(request.url,"},{"line_number":82,"context_line":"                                  headers\u003dheaders)"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"bf659307_5a9b5171","line":80,"in_reply_to":"bf659307_acd93b23","updated":"2018-03-26 05:50:57.000000000","message":"Done","commit_id":"c112f92bb28c157757cfe5f4b3c3db3b55235d31"}]}
