)]}'
{"keystone/middleware/auth_token.py":[{"author":{"_account_id":5,"name":"Ziad Sawalha","email":"ziad.sawalha@rackspace.com","username":"ziad-sawalha"},"change_message_id":"a948138c2d4fe9bed9f021419eff897acb15957f","unresolved":false,"context_lines":[{"line_number":153,"context_line":"        initialization code that applies to all protocols"},{"line_number":154,"context_line":"        \"\"\""},{"line_number":155,"context_line":"        LOG.info(\"Starting the %s component\", PROTOCOL_NAME)"},{"line_number":156,"context_line":"        print \"Starting the %s component\" % PROTOCOL_NAME"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"        #if app is set, then we are in a WSGI pipeline and requests get passed"},{"line_number":159,"context_line":"        # on to app. If it is not set, this component should forward requests"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADH%2F%2F%2Byc%3D","side":"PARENT","line":156,"updated":"2012-01-04 00:24:57.000000000","message":"If we remove this, we should remove it from all middleware.\n\nAlso, I know I rely on seeing this when I start a server. Not sure if others do, but would prefer more input from other projects before changing it in all middleware (including Keystone front ends).","commit_id":"b3c8111b1a742b10eb7c3447b0e3c9920ac57c62"},{"author":{"_account_id":5,"name":"Ziad Sawalha","email":"ziad.sawalha@rackspace.com","username":"ziad-sawalha"},"change_message_id":"a012322a1d2422febedc023570771358f3cdc943","unresolved":false,"context_lines":[{"line_number":153,"context_line":"        initialization code that applies to all protocols"},{"line_number":154,"context_line":"        \"\"\""},{"line_number":155,"context_line":"        LOG.info(\"Starting the %s component\", PROTOCOL_NAME)"},{"line_number":156,"context_line":"        print \"Starting the %s component\" % PROTOCOL_NAME"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"        #if app is set, then we are in a WSGI pipeline and requests get passed"},{"line_number":159,"context_line":"        # on to app. If it is not set, this component should forward requests"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADH%2F%2F%2BeE%3D","side":"PARENT","line":156,"in_reply_to":"AAAADH%2F%2F%2Bkg%3D","updated":"2012-01-04 19:27:18.000000000","message":"You\u0027ll get much more output than what is currently there, though. Logging is a different concern than interactive output. Using logging for output is not the correct approach (unless you\u0027re debugging, troubleshooting).","commit_id":"b3c8111b1a742b10eb7c3447b0e3c9920ac57c62"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"611481ee05dce4d1b4882766438cb373e00d4440","unresolved":false,"context_lines":[{"line_number":153,"context_line":"        initialization code that applies to all protocols"},{"line_number":154,"context_line":"        \"\"\""},{"line_number":155,"context_line":"        LOG.info(\"Starting the %s component\", PROTOCOL_NAME)"},{"line_number":156,"context_line":"        print \"Starting the %s component\" % PROTOCOL_NAME"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"        #if app is set, then we are in a WSGI pipeline and requests get passed"},{"line_number":159,"context_line":"        # on to app. If it is not set, this component should forward requests"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADH%2F%2F%2Bkg%3D","side":"PARENT","line":156,"in_reply_to":"AAAADH%2F%2F%2Byc%3D","updated":"2012-01-04 16:25:25.000000000","message":"Then configure your logging to route INFO to stdout!","commit_id":"b3c8111b1a742b10eb7c3447b0e3c9920ac57c62"},{"author":{"_account_id":5,"name":"Ziad Sawalha","email":"ziad.sawalha@rackspace.com","username":"ziad-sawalha"},"change_message_id":"a948138c2d4fe9bed9f021419eff897acb15957f","unresolved":false,"context_lines":[{"line_number":198,"context_line":"                                        self.auth_port))"},{"line_number":199,"context_line":"        LOG.debug(\"Authentication Service:%s\", self.auth_location)"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"        "},{"line_number":202,"context_line":"        # Credentials used to verify this component with the Auth service since"},{"line_number":203,"context_line":"        # validating tokens is a privileged call"},{"line_number":204,"context_line":"        self.admin_token \u003d conf.get(\u0027admin_token\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADH%2F%2F%2Byg%3D","line":201,"updated":"2012-01-04 00:24:57.000000000","message":"pep8","commit_id":"ba8a7dd271a3d54dd1939664f5e56a920ed0f875"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"611481ee05dce4d1b4882766438cb373e00d4440","unresolved":false,"context_lines":[{"line_number":353,"context_line":"            if \"timeout\" in cache.set.func_code.co_varnames:"},{"line_number":354,"context_line":"                # swift cache"},{"line_number":355,"context_line":"#                expires \u003d self._convert_date(claims[\u0027expires\u0027])"},{"line_number":356,"context_line":"                #BUG"},{"line_number":357,"context_line":"                expires \u003d claims[\u0027expires\u0027]"},{"line_number":358,"context_line":"                claims \u003d self._protect_claims(token, claims)"},{"line_number":359,"context_line":"                cache.set(key, (claims, expires, valid),"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADH%2F%2F%2Bkc%3D","line":356,"updated":"2012-01-04 16:25:25.000000000","message":"\"#BUG\" isn\u0027t very helpful - either fix it or document what you can. No need to leave broken code commented out, either.","commit_id":"ba8a7dd271a3d54dd1939664f5e56a920ed0f875"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"611481ee05dce4d1b4882766438cb373e00d4440","unresolved":false,"context_lines":[{"line_number":428,"context_line":"                raise ValidationFailed()"},{"line_number":429,"context_line":"            if datetime.strptime(expires, EXPIRE_TIME_FORMAT) \u003c\u003d datetime.now():"},{"line_number":430,"context_line":"                raise TokenExpired()"},{"line_number":431,"context_line":"                LOG.debug(\"Claims (token) expired (according to cache)\")"},{"line_number":432,"context_line":"            return claims"},{"line_number":433,"context_line":""},{"line_number":434,"context_line":"        # Step 1: We need to auth with the keystone service, so get an"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADH%2F%2F%2BkY%3D","line":431,"updated":"2012-01-04 16:25:25.000000000","message":"This log statement is now unreachable?","commit_id":"ba8a7dd271a3d54dd1939664f5e56a920ed0f875"}]}
