)]}'
{"openstackclient/common/client_config.py":[{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"ee63f2c8ff389915c21541780b1382471d3111d5","unresolved":false,"context_lines":[{"line_number":181,"context_line":"        config \u003d self._auth_v2_ignore_v3(config)"},{"line_number":182,"context_line":"        config \u003d self._auth_default_domain(config)"},{"line_number":183,"context_line":""},{"line_number":184,"context_line":"        if LOG.isEnabledFor(logging.DEBUG):"},{"line_number":185,"context_line":"            LOG.debug(\"auth_config_hook(): %s\","},{"line_number":186,"context_line":"                      strutils.mask_password(six.text_type(config)))"},{"line_number":187,"context_line":"        return config"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_19092e96","line":184,"range":{"start_line":184,"start_character":11,"end_line":184,"end_character":43},"updated":"2016-10-11 11:43:28.000000000","message":"Is this necessary ?  Many other places logging debug message are not checking this.","commit_id":"a37addb7b034105ce03ee35a0a1610d191c4b269"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"4147dc8929f64cfb2fb4efa80d46321ab1e8ec5a","unresolved":false,"context_lines":[{"line_number":181,"context_line":"        config \u003d self._auth_v2_ignore_v3(config)"},{"line_number":182,"context_line":"        config \u003d self._auth_default_domain(config)"},{"line_number":183,"context_line":""},{"line_number":184,"context_line":"        if LOG.isEnabledFor(logging.DEBUG):"},{"line_number":185,"context_line":"            LOG.debug(\"auth_config_hook(): %s\","},{"line_number":186,"context_line":"                      strutils.mask_password(six.text_type(config)))"},{"line_number":187,"context_line":"        return config"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_2f5ad24f","line":184,"range":{"start_line":184,"start_character":11,"end_line":184,"end_character":43},"in_reply_to":"fa6399be_19092e96","updated":"2016-10-12 22:35:33.000000000","message":"First, this is a cherry pick, so if you don\u0027t want this it\u0027d have to be changed on master first.\n\nSecond, it\u0027s not required, but it\u0027s used here to avoid going through what might be the unnecessary work of stringifying the config object and then masking it\u0027s values.","commit_id":"a37addb7b034105ce03ee35a0a1610d191c4b269"}]}