)]}'
{"keystone/routers/service.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"ae44a8a1293abea5a322197fa5033b2627f246d8","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        mapper.connect(\"/ec2tokens\", controller\u003dauth_controller,"},{"line_number":46,"context_line":"                       action\u003d\"authenticate_ec2\","},{"line_number":47,"context_line":"                       conditions\u003ddict(method\u003d[\"POST\"]))"},{"line_number":48,"context_line":"        # TODO(yosshy): this should be deprecated"},{"line_number":49,"context_line":"        mapper.connect(\"/s3tokens\", controller\u003dauth_controller,"},{"line_number":50,"context_line":"                       action\u003d\"authenticate_s3\","},{"line_number":51,"context_line":"                       conditions\u003ddict(method\u003d[\"POST\"]))"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAD3%2F%2F7tg%3D","line":48,"updated":"2012-01-22 04:48:06.000000000","message":"A router change shouldn\u0027t be required since it\u0027s supported on /tokens, right?","commit_id":"e1ce63210826e7b0ec39b1672ee20889fc5fed63"},{"author":{"_account_id":659,"name":"Akira Yoshiyama","email":"akirayoshiyama@gmail.com","username":"yosshy"},"change_message_id":"db27944866c57ee8f5d624f6374150d3b43baf70","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        mapper.connect(\"/ec2tokens\", controller\u003dauth_controller,"},{"line_number":46,"context_line":"                       action\u003d\"authenticate_ec2\","},{"line_number":47,"context_line":"                       conditions\u003ddict(method\u003d[\"POST\"]))"},{"line_number":48,"context_line":"        # TODO(yosshy): this should be deprecated"},{"line_number":49,"context_line":"        mapper.connect(\"/s3tokens\", controller\u003dauth_controller,"},{"line_number":50,"context_line":"                       action\u003d\"authenticate_s3\","},{"line_number":51,"context_line":"                       conditions\u003ddict(method\u003d[\"POST\"]))"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAEH%2F%2F%2F8E%3D","line":48,"in_reply_to":"AAAAD3%2F%2F7tg%3D","updated":"2012-01-23 11:15:46.000000000","message":"I think /ec2tokens is the same with it.\nI want to remove /s3tokens when /ec2tokens is removed because these capabilities should be synchronized.","commit_id":"e1ce63210826e7b0ec39b1672ee20889fc5fed63"},{"author":{"_account_id":5,"name":"Ziad Sawalha","email":"ziad.sawalha@rackspace.com","username":"ziad-sawalha"},"change_message_id":"1c08eab1f51dcee08517299aac188300104ec0c0","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        mapper.connect(\"/ec2tokens\", controller\u003dauth_controller,"},{"line_number":46,"context_line":"                       action\u003d\"authenticate_ec2\","},{"line_number":47,"context_line":"                       conditions\u003ddict(method\u003d[\"POST\"]))"},{"line_number":48,"context_line":"        # TODO(yosshy): this should be deprecated"},{"line_number":49,"context_line":"        mapper.connect(\"/s3tokens\", controller\u003dauth_controller,"},{"line_number":50,"context_line":"                       action\u003d\"authenticate_s3\","},{"line_number":51,"context_line":"                       conditions\u003ddict(method\u003d[\"POST\"]))"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAEH%2F%2F%2F5c%3D","line":48,"in_reply_to":"AAAAEH%2F%2F%2F8E%3D","updated":"2012-01-23 15:12:59.000000000","message":"The only reason /ec2tokens is there is because some clients already use it. Anyone modifying a client (to use s3, for example),. should remove the old ec2 code.\nWe should not introduce a now divergence from the core contract with this.","commit_id":"e1ce63210826e7b0ec39b1672ee20889fc5fed63"}],"keystone/test/unit/test_authn_s3.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"ae44a8a1293abea5a322197fa5033b2627f246d8","unresolved":false,"context_lines":[{"line_number":269,"context_line":"        # Delete the \u0027bad\u0027 tenant, orphaning the creds"},{"line_number":270,"context_line":"        self.get_request(\u0027DELETE\u0027, \u0027/tenants/bad\u0027)"},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"        url \u003d \"/s3tokens\""},{"line_number":273,"context_line":"        req \u003d self.get_request(\u0027POST\u0027, url)"},{"line_number":274,"context_line":"        params \u003d {"},{"line_number":275,"context_line":"            \"x-amz-acl\": \"public-read-write\","}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAD3%2F%2F7tk%3D","line":272,"updated":"2012-01-22 04:48:06.000000000","message":"/tokens?","commit_id":"e1ce63210826e7b0ec39b1672ee20889fc5fed63"},{"author":{"_account_id":659,"name":"Akira Yoshiyama","email":"akirayoshiyama@gmail.com","username":"yosshy"},"change_message_id":"db27944866c57ee8f5d624f6374150d3b43baf70","unresolved":false,"context_lines":[{"line_number":269,"context_line":"        # Delete the \u0027bad\u0027 tenant, orphaning the creds"},{"line_number":270,"context_line":"        self.get_request(\u0027DELETE\u0027, \u0027/tenants/bad\u0027)"},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"        url \u003d \"/s3tokens\""},{"line_number":273,"context_line":"        req \u003d self.get_request(\u0027POST\u0027, url)"},{"line_number":274,"context_line":"        params \u003d {"},{"line_number":275,"context_line":"            \"x-amz-acl\": \"public-read-write\","}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAEH%2F%2F%2F8I%3D","line":272,"in_reply_to":"AAAAD3%2F%2F7tk%3D","updated":"2012-01-23 11:15:46.000000000","message":"The same as test_authn_ec2.py.\nSee below:\nhttps://github.com/openstack/keystone/blob/master/keystone/test/unit/test_authn_ec2.py#L268","commit_id":"e1ce63210826e7b0ec39b1672ee20889fc5fed63"},{"author":{"_account_id":5,"name":"Ziad Sawalha","email":"ziad.sawalha@rackspace.com","username":"ziad-sawalha"},"change_message_id":"1c08eab1f51dcee08517299aac188300104ec0c0","unresolved":false,"context_lines":[{"line_number":269,"context_line":"        # Delete the \u0027bad\u0027 tenant, orphaning the creds"},{"line_number":270,"context_line":"        self.get_request(\u0027DELETE\u0027, \u0027/tenants/bad\u0027)"},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"        url \u003d \"/s3tokens\""},{"line_number":273,"context_line":"        req \u003d self.get_request(\u0027POST\u0027, url)"},{"line_number":274,"context_line":"        params \u003d {"},{"line_number":275,"context_line":"            \"x-amz-acl\": \"public-read-write\","}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAEH%2F%2F%2F5U%3D","line":272,"in_reply_to":"AAAAEH%2F%2F%2F8I%3D","updated":"2012-01-23 15:12:59.000000000","message":"Same comment. The need for us to maintain /ec2tokens for compatibility does not exist for /s3tokens.","commit_id":"e1ce63210826e7b0ec39b1672ee20889fc5fed63"}]}