)]}'
{"keystoneclient/session.py":[{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"053ce773433853f6970b5c20a4a95eeee2f37525","unresolved":false,"context_lines":[{"line_number":134,"context_line":"                if http_cache:"},{"line_number":135,"context_line":"                    session.mount(scheme, CachingAdapter(cache\u003dhttp_cache))"},{"line_number":136,"context_line":"                else:"},{"line_number":137,"context_line":"                    session.mount(scheme, TCPKeepAliveAdapter())"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"        self.auth \u003d auth"},{"line_number":140,"context_line":"        self.session \u003d session"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a4dcd0f_532aad91","line":137,"updated":"2015-08-11 02:20:20.000000000","message":"why this as opposed to: \n\nif http_cache:\n   session \u003d cachecontrol.CacheControl(session) \n\nwhat does the Adapter buy us?","commit_id":"f538cc628195a808fdf015432da695ea131e6595"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"5b6a0ecbfc921c98bf278945fe5e6637a476a25d","unresolved":false,"context_lines":[{"line_number":134,"context_line":"                if http_cache:"},{"line_number":135,"context_line":"                    session.mount(scheme, CachingAdapter(cache\u003dhttp_cache))"},{"line_number":136,"context_line":"                else:"},{"line_number":137,"context_line":"                    session.mount(scheme, TCPKeepAliveAdapter())"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"        self.auth \u003d auth"},{"line_number":140,"context_line":"        self.session \u003d session"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a4dcd0f_ba2bc1ea","line":137,"in_reply_to":"1a4dcd0f_532aad91","updated":"2015-08-11 03:28:41.000000000","message":"requests only allows one adapter per scheme. If I used this https://github.com/ionrock/cachecontrol/blob/master/cachecontrol/wrapper.py I wouldn\u0027t be able to keep the TCPKeepAliveAdapter.","commit_id":"f538cc628195a808fdf015432da695ea131e6595"},{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"ac4ab5d07b328bbe1bb7b3be960b4812b8966ec7","unresolved":false,"context_lines":[{"line_number":134,"context_line":"                if http_cache:"},{"line_number":135,"context_line":"                    session.mount(scheme, CachingAdapter(cache\u003dhttp_cache))"},{"line_number":136,"context_line":"                else:"},{"line_number":137,"context_line":"                    session.mount(scheme, TCPKeepAliveAdapter())"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"        self.auth \u003d auth"},{"line_number":140,"context_line":"        self.session \u003d session"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a4dcd0f_5a507d5e","line":137,"in_reply_to":"1a4dcd0f_ba2bc1ea","updated":"2015-08-11 03:33:47.000000000","message":"oh - right, that\u0027s sneaky. I expected this to be a class that wrapped the whole session object not a capitalized function that replaced adapters on the original.","commit_id":"f538cc628195a808fdf015432da695ea131e6595"}],"requirements.txt":[{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"053ce773433853f6970b5c20a4a95eeee2f37525","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"# HTTP cache support"},{"line_number":22,"context_line":"lockfile\u003d\u003d0.10.2"},{"line_number":23,"context_line":"CacheControl\u003d\u003d0.11.5 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":1,"id":"1a4dcd0f_b7ff5c00","line":23,"updated":"2015-08-11 02:20:20.000000000","message":"I\u0027m not sure how i feel about a hard dep on these, particularly given keystoneauth is supposed to be lightweight.","commit_id":"f538cc628195a808fdf015432da695ea131e6595"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"5b6a0ecbfc921c98bf278945fe5e6637a476a25d","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"# HTTP cache support"},{"line_number":22,"context_line":"lockfile\u003d\u003d0.10.2"},{"line_number":23,"context_line":"CacheControl\u003d\u003d0.11.5 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":1,"id":"1a4dcd0f_9a38453f","line":23,"in_reply_to":"1a4dcd0f_b7ff5c00","updated":"2015-08-11 03:28:41.000000000","message":"We can probably make these optional like we are doing in Keystone. On the other hand caching seems like the right thing to start getting people to use.","commit_id":"f538cc628195a808fdf015432da695ea131e6595"}]}