)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c7dcc4bf831eba81e8a1ab56892ee22d7bd435a1","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Tobias Urdin \u003ctobias.urdin@binero.se\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-05-25 09:33:10 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"When Nova-Api runs behind the load balancer or Reverse proxy,"},{"line_number":8,"context_line":"Loadbalancer IP is getting logged in nova_api.log instead of end user source ip by RequestLog"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"It should check for CONF.api.use_forwarded_for and"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"03f86837_3d3580ba","line":7,"updated":"2021-06-01 11:37:33.000000000","message":"This needs a summary line, please. Something like:\n\n  api: Log correct client IP if load balancer in use","commit_id":"ff9b88154e456b98ebec7c838d1e9a6f3ae275b4"}],"nova/api/openstack/requestlog.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c7dcc4bf831eba81e8a1ab56892ee22d7bd435a1","unresolved":true,"context_lines":[{"line_number":71,"context_line":""},{"line_number":72,"context_line":"        remote_address \u003d req.environ.get(\u0027REMOTE_ADDR\u0027, \u0027-\u0027)"},{"line_number":73,"context_line":"        if CONF.api.use_forwarded_for:"},{"line_number":74,"context_line":"            remote_address \u003d req.environ.get(\u0027HTTP_X_FORWARDED_FOR\u0027,"},{"line_number":75,"context_line":"                                             remote_address"},{"line_number":76,"context_line":"                                            )"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        data \u003d {"},{"line_number":79,"context_line":"            \u0027REMOTE_ADDR\u0027: remote_address,"}],"source_content_type":"text/x-python","patch_set":4,"id":"f217488d_55836514","line":76,"range":{"start_line":74,"start_character":0,"end_line":76,"end_character":45},"updated":"2021-06-01 11:37:33.000000000","message":"style nit: Can you reflow this slightly?\n\n  remote_address \u003d req.environ.get(\n      \u0027HTTP_X_FORWARDED_FOR\u0027, remote_address)\n\na comment about what you\u0027re doing would be helpful also, as would a simple test in [1]\n\n[1] nova/tests/unit/api/openstack/test_requestlog.py","commit_id":"ff9b88154e456b98ebec7c838d1e9a6f3ae275b4"}]}
