)]}'
{"swift/common/middleware/s3api/s3api.py":[{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"6a8f4b9334d976f7ee1c980bcc7e2ba7dff35bfe","unresolved":true,"context_lines":[{"line_number":315,"context_line":"        # Not S3, apparently"},{"line_number":316,"context_line":"        return False"},{"line_number":317,"context_line":""},{"line_number":318,"context_line":"    def _emit_header_stats(self, req):"},{"line_number":319,"context_line":"        headers2stats_bucket \u003d ["},{"line_number":320,"context_line":"            # ((headers to match), increment_bucket_stat),"},{"line_number":321,"context_line":"            ((\u0027Content-MD5\u0027,), \u0027header.s3.content.md5\u0027),"}],"source_content_type":"text/x-python","patch_set":1,"id":"f6fe7755_ca7488cd","line":318,"updated":"2025-01-23 06:44:24.000000000","message":"Not sure if this covers them all. There are alot of headers we could check.","commit_id":"5d4996305122acda8adb4ba22046111266d09eb3"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"6a8f4b9334d976f7ee1c980bcc7e2ba7dff35bfe","unresolved":true,"context_lines":[{"line_number":373,"context_line":"        try:"},{"line_number":374,"context_line":"            req_class \u003d get_request_class(env, self.conf.s3_acl)"},{"line_number":375,"context_line":"            req \u003d req_class(env, self.app, self.conf)"},{"line_number":376,"context_line":"            self._emit_header_stats(req)"},{"line_number":377,"context_line":"            resp \u003d self.handle_request(req)"},{"line_number":378,"context_line":"        except NotS3Request:"},{"line_number":379,"context_line":"            resp \u003d self.app"}],"source_content_type":"text/x-python","patch_set":1,"id":"14c2530d_95b7ff5b","line":376,"updated":"2025-01-23 06:44:24.000000000","message":"I added this quite early in s3api to minimise code.. but does mean it gets run on every request.\n\nI could move this to the finally in proxy logging so it\u0027s emitted after the user gets their data, and it wont effect traffic so much :hmm:","commit_id":"5d4996305122acda8adb4ba22046111266d09eb3"}]}
