)]}'
{"swift/common/middleware/proxy_logging.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"8e19863fb692042b04d3ea5ffdf864f28feb9d54","unresolved":true,"context_lines":[{"line_number":145,"context_line":"                    \u0027log_statsd_sample_rate_factor\u0027,"},{"line_number":146,"context_line":"                    \u0027log_statsd_metric_prefix\u0027,"},{"line_number":147,"context_line":"                    \u0027log_statsd_label_mode\u0027,"},{"line_number":148,"context_line":"                    \u0027log_statsd_emit_legacy\u0027):"},{"line_number":149,"context_line":"            value \u003d conf.get(\u0027access_\u0027 + key, conf.get(key, None))"},{"line_number":150,"context_line":"            if value:"},{"line_number":151,"context_line":"                access_log_conf[key] \u003d value"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a9e3e3a_8584beaa","line":148,"updated":"2024-03-26 19:25:33.000000000","message":"Ugh... do we want to carry forward this `access_log_...` cruft for the operator-defined labels? It feels a little weird that we do it for `label_mode`/`emit_legacy` but not that...","commit_id":"31038c3e1f573f6cf04f68e22fe950298983f7ef"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"8e19863fb692042b04d3ea5ffdf864f28feb9d54","unresolved":true,"context_lines":[{"line_number":465,"context_line":"            swift_path \u003d req.environ.get(\u0027swift.backend_path\u0027, req.path)"},{"line_number":466,"context_line":"            if swift_path.startswith(\u0027/v1/\u0027):"},{"line_number":467,"context_line":"                _, acc, cont, obj \u003d split_path(swift_path, 1, 4, True)"},{"line_number":468,"context_line":"            acc \u003d StrAnonymizer(acc, self.anonymization_method,"},{"line_number":469,"context_line":"                                self.anonymization_salt)"},{"line_number":470,"context_line":"            cont \u003d StrAnonymizer(cont, self.anonymization_method,"},{"line_number":471,"context_line":"                                 self.anonymization_salt)"}],"source_content_type":"text/x-python","patch_set":1,"id":"eba15362_46581f34","line":468,"range":{"start_line":468,"start_character":18,"end_line":468,"end_character":31},"updated":"2024-03-26 19:25:33.000000000","message":"This was a problem in my patch, too, but just wrapping the account in the `StrAnonymizer` doesn\u0027t get us what we want. Maybe it\u0027d be better for us to just drop the idea for now, and let a group like OVH (which wrote the log anonymization stuff to ensure compliance with GDPR) propose how we *should* address it.","commit_id":"31038c3e1f573f6cf04f68e22fe950298983f7ef"}]}