)]}'
{"keystoneclient/middleware/auth_token.py":[{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"cbe50b99d1749b4cc3ee12fd4079d0ff4e5b5e3f","unresolved":false,"context_lines":[{"line_number":919,"context_line":"        # with the official python memcache client with time\u003d since"},{"line_number":920,"context_line":"        # grizzly, we still need to handle folsom for a while until"},{"line_number":921,"context_line":"        # this could get removed."},{"line_number":922,"context_line":"        try:"},{"line_number":923,"context_line":"            self._cache.set(cache_key,"},{"line_number":924,"context_line":"                            data_to_store,"},{"line_number":925,"context_line":"                            time\u003dself.token_cache_time)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPn%2F%2Fz1E%3D","line":922,"updated":"2013-06-20 17:38:41.000000000","message":"yeah, looks like self._use_keystone_cache is no longer relevant. Please remove it.","commit_id":"cea40c356af31bb77f4424df730284927265ee59"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"b0b1f0a8ac7cebef5d2c75ddfb92d01f02be8838","unresolved":false,"context_lines":[{"line_number":921,"context_line":"            self._cache.set(cache_key,"},{"line_number":922,"context_line":"                            data_to_store,"},{"line_number":923,"context_line":"                            time\u003dself.token_cache_time)"},{"line_number":924,"context_line":"        except(TypeError):"},{"line_number":925,"context_line":"            self._cache.set(cache_key,"},{"line_number":926,"context_line":"                            data_to_store,"},{"line_number":927,"context_line":"                            timeout\u003dself.token_cache_time)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2FvX0%3D","line":924,"updated":"2013-06-23 16:25:55.000000000","message":"should this be a configuration option?\n\nIs there any way to introspect the function object to see what it supports, or tell the version by some other way (check for an attribute or something)?\n\nAlternatively, keep track of if this failed last time so don\u0027t try again.","commit_id":"19591a51e4f9fe5b10f3cf4146277f169786450f"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"b291f73e3c909e713441edaabd847d516eb063eb","unresolved":false,"context_lines":[{"line_number":921,"context_line":"            self._cache.set(cache_key,"},{"line_number":922,"context_line":"                            data_to_store,"},{"line_number":923,"context_line":"                            time\u003dself.token_cache_time)"},{"line_number":924,"context_line":"        except(TypeError):"},{"line_number":925,"context_line":"            self._cache.set(cache_key,"},{"line_number":926,"context_line":"                            data_to_store,"},{"line_number":927,"context_line":"                            timeout\u003dself.token_cache_time)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2FtyM%3D","line":924,"in_reply_to":"AAAAPn%2F%2Ftzk%3D","updated":"2013-06-24 16:28:32.000000000","message":"I agree. Current implementation is adequate.","commit_id":"19591a51e4f9fe5b10f3cf4146277f169786450f"},{"author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"change_message_id":"ff457c67b499e687e14a7afc35e606ad0cf1e68a","unresolved":false,"context_lines":[{"line_number":921,"context_line":"            self._cache.set(cache_key,"},{"line_number":922,"context_line":"                            data_to_store,"},{"line_number":923,"context_line":"                            time\u003dself.token_cache_time)"},{"line_number":924,"context_line":"        except(TypeError):"},{"line_number":925,"context_line":"            self._cache.set(cache_key,"},{"line_number":926,"context_line":"                            data_to_store,"},{"line_number":927,"context_line":"                            timeout\u003dself.token_cache_time)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2Ftzk%3D","line":924,"in_reply_to":"AAAAPn%2F%2FvX0%3D","updated":"2013-06-24 16:21:05.000000000","message":"Do you think this is really needed? I can add complexity to the code but that\u0027s only temporary (i.e: until we don\u0027t support folsom anymore in auth_token) and having a new configurable option is usually costly and can be confusing for end users. I can do the tracking thing but I am not sure if that bring any advantages from a code POV and arguably more complex. If you really feel strongly about this I would not mind changing it.","commit_id":"19591a51e4f9fe5b10f3cf4146277f169786450f"}]}