)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"ab0e7dbc45078b606b83904e51f40475a66c3c06","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Adds hacking check for debug logging translations"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"bp more-code-style-automation"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Id54f322f00b04a165bb4a7b1e24f95bb72b7f068"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"baada198_d94b6885","line":10,"updated":"2014-07-02 20:39:28.000000000","message":"Could we add this bug to the commit message: \n\nhttps://bugs.launchpad.net/keystone/+bug/1317831\n\nI see there was a discussion in https://review.openstack.org/#/c/93013/ about which review to go with. If we do end up going with this review and adding the hacking check, we should include that bug in the commit message.","commit_id":"76baf5b691063e8845b9e9c72d55f416208a34ad"}],"keystone/common/ldap/core.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"465b7f83ff2aa9a218d1ab20e6a47a56f4590e96","unresolved":false,"context_lines":[{"line_number":150,"context_line":"                          dict((kind, [ldap2py(x) for x in values])"},{"line_number":151,"context_line":"                               for kind, values in six.iteritems(attrs))))"},{"line_number":152,"context_line":"    if at_least_one_referral:"},{"line_number":153,"context_line":"        LOG.debug((\u0027Referrals were returned and ignored. Enable referral \u0027"},{"line_number":154,"context_line":"                   \u0027chasing in keystone.conf via [ldap] chase_referrals\u0027))"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    return py_result"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_45503e5e","line":153,"updated":"2014-06-26 01:53:17.000000000","message":"no need for double parentheses here?","commit_id":"0eaa4c1a9feddb996cfd6bafa1e42564501506f7"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"53c5d7b13c7b2effb61e68fa224cb663dbc45b15","unresolved":false,"context_lines":[{"line_number":150,"context_line":"                          dict((kind, [ldap2py(x) for x in values])"},{"line_number":151,"context_line":"                               for kind, values in six.iteritems(attrs))))"},{"line_number":152,"context_line":"    if at_least_one_referral:"},{"line_number":153,"context_line":"        LOG.debug((\u0027Referrals were returned and ignored. Enable referral \u0027"},{"line_number":154,"context_line":"                   \u0027chasing in keystone.conf via [ldap] chase_referrals\u0027))"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    return py_result"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_a72abc7b","line":153,"in_reply_to":"1ae5cdf2_45503e5e","updated":"2014-06-27 02:00:32.000000000","message":"This is a habit. I tend to surround adjacent strings in parens so others know it\u0027s intentional and I\u0027m not just missing a comma. If it looks better without I can remove them.","commit_id":"0eaa4c1a9feddb996cfd6bafa1e42564501506f7"}],"keystone/common/wsgi.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"465b7f83ff2aa9a218d1ab20e6a47a56f4590e96","unresolved":false,"context_lines":[{"line_number":316,"context_line":"        \"\"\""},{"line_number":317,"context_line":"        if (\u0027token_id\u0027 not in context or"},{"line_number":318,"context_line":"                context.get(\u0027token_id\u0027) \u003d\u003d CONF.admin_token):"},{"line_number":319,"context_line":"            LOG.debug((\u0027will not lookup trust as the request auth token is \u0027"},{"line_number":320,"context_line":"                       \u0027either absent or it is the system admin token\u0027))"},{"line_number":321,"context_line":"            return None"},{"line_number":322,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_05e9d67e","line":319,"updated":"2014-06-26 01:53:17.000000000","message":"no need for double parentheses here","commit_id":"0eaa4c1a9feddb996cfd6bafa1e42564501506f7"}],"keystone/middleware/core.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"465b7f83ff2aa9a218d1ab20e6a47a56f4590e96","unresolved":false,"context_lines":[{"line_number":273,"context_line":""},{"line_number":274,"context_line":"    def process_request(self, request):"},{"line_number":275,"context_line":"        if AUTH_TOKEN_HEADER not in request.headers:"},{"line_number":276,"context_line":"            LOG.debug((\u0027Auth token not in the request header. \u0027"},{"line_number":277,"context_line":"                       \u0027Will not build auth context.\u0027))"},{"line_number":278,"context_line":"            return"},{"line_number":279,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_a594a2e8","line":276,"updated":"2014-06-26 01:53:17.000000000","message":"no need for double parens here?","commit_id":"0eaa4c1a9feddb996cfd6bafa1e42564501506f7"}],"keystone/notifications.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"465b7f83ff2aa9a218d1ab20e6a47a56f4590e96","unresolved":false,"context_lines":[{"line_number":290,"context_line":""},{"line_number":291,"context_line":"    context \u003d {}"},{"line_number":292,"context_line":"    payload \u003d event.as_dict()"},{"line_number":293,"context_line":"    LOG.debug(\u0027CADF Event: %s\u0027, payload)"},{"line_number":294,"context_line":"    service \u003d \u0027identity\u0027"},{"line_number":295,"context_line":"    event_type \u003d \u0027%(service)s.%(action)s\u0027 % {\u0027service\u0027: service,"},{"line_number":296,"context_line":"                                             \u0027action\u0027: action}"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_658e1ad3","line":293,"updated":"2014-06-26 01:53:17.000000000","message":"hmm, these probably shouldn\u0027t be debugs? (i know this is a completely separate topic, just noting this for my own record)","commit_id":"0eaa4c1a9feddb996cfd6bafa1e42564501506f7"}]}