)]}'
{"glance/common/utils.py":[{"author":{"_account_id":4463,"name":"iccha-sethi","email":"iccha.sethi@rackspace.com","username":"iccha-sethi"},"change_message_id":"7b337315b4d5108942caa964e245d78c38545077","unresolved":false,"context_lines":[{"line_number":435,"context_line":"            raise"},{"line_number":436,"context_line":""},{"line_number":437,"context_line":""},{"line_number":438,"context_line":"def redact_loc(image_meta):"},{"line_number":439,"context_line":"    \"\"\""},{"line_number":440,"context_line":"    Create a shallow copy of image meta with \u0027location\u0027 redacted"},{"line_number":441,"context_line":"    for security (as it can contain credentials)."}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Bxzg%3D","line":438,"updated":"2013-02-19 14:06:28.000000000","message":"Can we make this a more generic function where we can redact the parameter/s we pass in from image meta?","commit_id":"797c2a2909c130951b0765201bae4e2f04bb995c"},{"author":{"_account_id":4463,"name":"iccha-sethi","email":"iccha.sethi@rackspace.com","username":"iccha-sethi"},"change_message_id":"dbcc3cf5ef8654923e126bc9045f0652120a4d59","unresolved":false,"context_lines":[{"line_number":435,"context_line":"            raise"},{"line_number":436,"context_line":""},{"line_number":437,"context_line":""},{"line_number":438,"context_line":"def redact_loc(image_meta):"},{"line_number":439,"context_line":"    \"\"\""},{"line_number":440,"context_line":"    Create a shallow copy of image meta with \u0027location\u0027 redacted"},{"line_number":441,"context_line":"    for security (as it can contain credentials)."}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2BxtY%3D","line":438,"in_reply_to":"AAAAM3%2F%2BxuQ%3D","updated":"2013-02-19 14:34:12.000000000","message":"sounds reasonable","commit_id":"797c2a2909c130951b0765201bae4e2f04bb995c"},{"author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"change_message_id":"6782903c62026729fbc7488d3bd7b86daaa826ab","unresolved":false,"context_lines":[{"line_number":435,"context_line":"            raise"},{"line_number":436,"context_line":""},{"line_number":437,"context_line":""},{"line_number":438,"context_line":"def redact_loc(image_meta):"},{"line_number":439,"context_line":"    \"\"\""},{"line_number":440,"context_line":"    Create a shallow copy of image meta with \u0027location\u0027 redacted"},{"line_number":441,"context_line":"    for security (as it can contain credentials)."}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2BxuQ%3D","line":438,"in_reply_to":"AAAAM3%2F%2Bxzg%3D","updated":"2013-02-19 14:22:29.000000000","message":"Hi iccha, thanks for your review. I\u0027m not sure there\u0027s a need to make this generic right now. What\u0027s there is in the spirit of RegistryClient.encrypt_metadata, which just targets location. Personally I\u0027d lean towards making it generic once the need arises, ie once we require redaction of some other parameter. What do you think?","commit_id":"797c2a2909c130951b0765201bae4e2f04bb995c"}]}