)]}'
{"id":"openstack%2Fpython-keystoneclient~278027","triplet_id":"openstack%2Fpython-keystoneclient~master~Ic365c654ebca4045208c6c30e232665145db7b4c","project":"openstack/python-keystoneclient","branch":"master","topic":"bug/1453953","hashtags":[],"change_id":"Ic365c654ebca4045208c6c30e232665145db7b4c","subject":"Handle exception on UnicodeDecodError in logging of request","status":"MERGED","created":"2016-02-09 18:44:57.000000000","updated":"2016-02-12 04:37:43.000000000","submitted":"2016-02-12 04:37:43.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":24,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"1da5311018ceb7448411dead1604710ac3f97dee","_number":278027,"virtual_id_number":278027,"owner":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},{"value":2,"date":"2016-02-12 04:37:43.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},{"value":0,"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},{"value":0,"_account_id":8090,"name":"Chad Roberts","email":"croberts@redhat.com","username":"croberts"},{"value":0,"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},{"value":0,"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},{"value":0,"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"all":[{"value":2,"date":"2016-02-12 03:41:30.000000000","_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":1,"date":"2016-02-10 19:29:49.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2016-02-11 06:10:54.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},{"value":2,"date":"2016-02-12 03:46:36.000000000","_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},{"value":1,"date":"2016-02-11 15:32:03.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":8090,"name":"Chad Roberts","email":"croberts@redhat.com","username":"croberts"},{"value":0,"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},{"value":0,"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},{"value":1,"date":"2016-02-11 13:43:06.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"all":[{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},{"value":1,"date":"2016-02-12 03:46:36.000000000","_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},{"value":0,"_account_id":8090,"name":"Chad Roberts","email":"croberts@redhat.com","username":"croberts"},{"value":0,"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},{"value":0,"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},{"value":0,"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},{"_account_id":8090,"name":"Chad Roberts","email":"croberts@redhat.com","username":"croberts"},{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-02-09 19:06:19.000000000","updated_by":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"reviewer":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"state":"REVIEWER"},{"updated":"2016-02-10 19:29:49.000000000","updated_by":{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},"reviewer":{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},"state":"REVIEWER"},{"updated":"2016-02-11 06:10:54.000000000","updated_by":{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},"reviewer":{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},"state":"REVIEWER"},{"updated":"2016-02-11 13:43:06.000000000","updated_by":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"reviewer":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"state":"REVIEWER"},{"updated":"2016-02-11 15:32:03.000000000","updated_by":{"_account_id":8090,"name":"Chad Roberts","email":"croberts@redhat.com","username":"croberts"},"reviewer":{"_account_id":8090,"name":"Chad Roberts","email":"croberts@redhat.com","username":"croberts"},"state":"REVIEWER"},{"updated":"2016-02-12 03:41:30.000000000","updated_by":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"reviewer":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"state":"REVIEWER"},{"updated":"2016-02-12 03:46:36.000000000","updated_by":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"reviewer":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"state":"REVIEWER"},{"updated":"2016-02-12 04:37:43.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"4ed1465992445effbb828fc80e41e576eb753231","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-09 18:44:57.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"e78afb82b3b4945e93f4371a33b96cc4cdb5015d","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-09 18:47:40.000000000","message":"Patch Set 1:\n\nHi folks.  This an attempt to revisit the issue that was abandoned in https://review.openstack.org/#/c/183514/\n\nIt\u0027s a simple approach, only trying to fix on fail so Python version shouldn\u0027t matter. It tries to target the failure specifically to the data payload, and it uses a suggestion from the previous review to decode and replace errant characters.","accounts_in_message":[],"_revision_number":1},{"id":"2e444803925a9882d28e6cdb76192b3ed1fe5e54","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-09 18:51:26.000000000","message":"Patch Set 1:\n\nPS, if people like the approach, I\u0027ll add in the test from the other review. Also note that this patch only touches the _http_log_response method, whereas the initial review also handled _positive_nonzero_float","accounts_in_message":[],"_revision_number":1},{"id":"59a5a85a43b1785f81d58c8ae322f9156f8e8305","author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"date":"2016-02-09 19:02:38.000000000","message":"Patch Set 1:\n\n(1 comment)\n\ni have a minor nit with the log message","accounts_in_message":[],"_revision_number":1},{"id":"c85fec94a838e0dd3f0270abbd3cbd3d8fd6431c","author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"date":"2016-02-09 19:06:19.000000000","message":"Patch Set 1: Code-Review-1\n\n(2 comments)\n\nalso missing a test proving that this fixes the bug.","accounts_in_message":[],"_revision_number":1},{"id":"4718f583ba9b8632509f1dac8d9ed5e901360505","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-09 19:18:22.000000000","message":"Patch Set 1:\n\n(4 comments)\n\nthanks guys.  Working on pulling the test over now.","accounts_in_message":[],"_revision_number":1},{"id":"d25b10b804f4c87356ef9c32d21c20c02b438799","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-09 19:28:26.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"81a75075e6ec4952839363752b743813b900ba38","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-09 19:29:30.000000000","message":"Patch Set 2:\n\nmuch, much simpler.  What do you think?","accounts_in_message":[],"_revision_number":2},{"id":"00edcc302f756c3f8bbfa3b90240df91bd83d915","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-09 19:57:09.000000000","message":"Patch Set 2: Workflow-1\n\nneeds a tweak, based on real life ...","accounts_in_message":[],"_revision_number":2},{"id":"3ea6794653e89b3d5b212cb674f46a78945b179f","author":{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},"date":"2016-02-09 19:57:29.000000000","message":"Patch Set 2: Code-Review-1\n\n(3 comments)","accounts_in_message":[],"_revision_number":2},{"id":"dd6c5a757baea09b37c4f5d48a359b2268839355","author":{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},"date":"2016-02-09 19:57:38.000000000","message":"Patch Set 2:\n\nsorry i reviewed it before you workflowed it :(","accounts_in_message":[],"_revision_number":2},{"id":"9272ae1a17120d9c8aa2dfd52f54dbf6b0b2b0fc","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-09 20:03:56.000000000","message":"Patch Set 2:\n\n@Tom, no problem at all, thanks for the comment","accounts_in_message":[],"_revision_number":2},{"id":"91ce6243e96ade762fb639549bc06a589b52c413","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-09 20:23:27.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"a15d99e2da8422a04206d121385238ba0fb5a6cc","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-09 20:30:27.000000000","message":"Patch Set 4: Published edit on patch set 3","accounts_in_message":[],"_revision_number":4},{"id":"4ed8ad1e3b24a7fef786ccdc61fbd60350bdd12b","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-09 20:32:08.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"944a181b635b745c6bbe094be412cc1ec138cc1f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-02-09 22:44:50.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-keystoneclient-docs http://docs-draft.openstack.org/27/278027/4/check/gate-python-keystoneclient-docs/65a5007//doc/build/html/ : SUCCESS in 42s\n- gate-python-keystoneclient-linters http://logs.openstack.org/27/278027/4/check/gate-python-keystoneclient-linters/bcbdcfd/ : SUCCESS in 41s\n- gate-python-keystoneclient-python27 http://logs.openstack.org/27/278027/4/check/gate-python-keystoneclient-python27/5b30f45/ : SUCCESS in 1m 15s\n- gate-python-keystoneclient-python34 http://logs.openstack.org/27/278027/4/check/gate-python-keystoneclient-python34/770f673/ : SUCCESS in 3m 32s\n- gate-tempest-dsvm-neutron-src-python-keystoneclient http://logs.openstack.org/27/278027/4/check/gate-tempest-dsvm-neutron-src-python-keystoneclient/55ef187/ : SUCCESS in 1h 04m 05s\n- gate-keystoneclient-dsvm-functional http://logs.openstack.org/27/278027/4/check/gate-keystoneclient-dsvm-functional/1085838/ : SUCCESS in 25m 01s","accounts_in_message":[],"_revision_number":4},{"id":"e45ad8f1080b22dff42fdde2dd97dea2061bf7b0","author":{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},"date":"2016-02-09 22:57:10.000000000","message":"Patch Set 4: Code-Review+1\n\n(4 comments)\n\nlooking good.  just some tweak to the look of it. but other than that this is gold. :-)","accounts_in_message":[],"_revision_number":4},{"id":"fd55e7272aa555adc16a1872ba66250b280cb836","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-10 15:49:21.000000000","message":"Patch Set 4:\n\n(3 comments)","accounts_in_message":[],"_revision_number":4},{"id":"947784cece4ad0851c599aa2d7228420d76066d1","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-10 17:15:45.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"88051c035ac70e4588cf91d0657b950280061d12","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-10 17:38:51.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"c02a0e2adfdfcf544b0fb9202384c608ac94ae92","author":{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},"date":"2016-02-10 17:42:00.000000000","message":"Patch Set 4:\n\n(2 comments)","accounts_in_message":[],"_revision_number":4},{"id":"33fa8f20bdf34ab23e0835c44cb7fce676c3c748","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-10 17:45:04.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"3d747955528c754c2b7a3de11be28bf21fd07327","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-10 17:45:58.000000000","message":"Patch Set 5:\n\n@Tom, no problem! Thanks for the comments.","accounts_in_message":[],"_revision_number":5},{"id":"9a1ad2e6e896ad354d821aed8156d3b5f893c82d","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-10 17:49:06.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"7114c32c9c3eb3b132d0fddc07e20a59212eafe0","author":{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},"date":"2016-02-10 17:54:19.000000000","message":"Patch Set 7: Code-Review+1\n\n(2 comments)\n\nlooks good. sorry i didn\u0027t think about this sooner.  could you check to make sure the string with the \u0027replaced characters\u0027 made it into the log?","accounts_in_message":[],"_revision_number":7},{"id":"7008feee840ab872f74f4ecf909cb43c3ea7845c","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-10 19:04:56.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"298e9ed8529d6e095be5572937bc7525ecb24fee","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2016-02-10 19:06:58.000000000","message":"Patch Set 7:\n\n(1 comment)","accounts_in_message":[],"_revision_number":7},{"id":"00b5bb0ee8e4201b4eda717ca274b2b467296b98","author":{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},"date":"2016-02-10 19:29:49.000000000","message":"Patch Set 8: Code-Review+1\n\nMy concerns have been addressed. Thanks Trevor!","accounts_in_message":[],"_revision_number":8},{"id":"31e3f0706cc1adfe4195d7ee43a80b536995d4a0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-02-10 21:27:38.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-keystoneclient-docs http://docs-draft.openstack.org/27/278027/8/check/gate-python-keystoneclient-docs/c9a382b//doc/build/html/ : SUCCESS in 1m 13s\n- gate-python-keystoneclient-linters http://logs.openstack.org/27/278027/8/check/gate-python-keystoneclient-linters/24d7c15/ : SUCCESS in 45s\n- gate-python-keystoneclient-python27 http://logs.openstack.org/27/278027/8/check/gate-python-keystoneclient-python27/868ea1c/ : SUCCESS in 1m 37s\n- gate-python-keystoneclient-python34 http://logs.openstack.org/27/278027/8/check/gate-python-keystoneclient-python34/560ca98/ : SUCCESS in 3m 10s\n- gate-tempest-dsvm-neutron-src-python-keystoneclient http://logs.openstack.org/27/278027/8/check/gate-tempest-dsvm-neutron-src-python-keystoneclient/0bf9631/ : SUCCESS in 51m 48s\n- gate-keystoneclient-dsvm-functional http://logs.openstack.org/27/278027/8/check/gate-keystoneclient-dsvm-functional/4a6f9ea/ : SUCCESS in 16m 59s","accounts_in_message":[],"_revision_number":8},{"id":"d8f65cc42c1ea970cf46b4cf9ce24c51b4213261","author":{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},"date":"2016-02-11 06:10:54.000000000","message":"Patch Set 8: Code-Review+1","accounts_in_message":[],"_revision_number":8},{"id":"fb1a7bb4f4cef598f318b69fee75c5abf8f8cdb3","author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"date":"2016-02-11 13:43:06.000000000","message":"Patch Set 8: Code-Review+1","accounts_in_message":[],"_revision_number":8},{"id":"3c10be8c46d4d689e6d72fbd4d949d2a2257b724","author":{"_account_id":8090,"name":"Chad Roberts","email":"croberts@redhat.com","username":"croberts"},"date":"2016-02-11 15:32:03.000000000","message":"Patch Set 8: Code-Review+1","accounts_in_message":[],"_revision_number":8},{"id":"44c1f148c36f8e47a37cc28f701c07fabaf89288","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-02-12 03:41:30.000000000","message":"Patch Set 8: Code-Review+2","accounts_in_message":[],"_revision_number":8},{"id":"815e4681ceb12b3d405b7d93327634069ed944f9","author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"date":"2016-02-12 03:46:36.000000000","message":"Patch Set 8: Code-Review+2 Workflow+1\n\nSo I\u0027ll +A to get this moving along as it looks good. Can you please propose the same patch to keystoneauth where new development is happening? \n\nTrevor: If you\u0027re ok, i think the solution here is to be less clever. We can change the whole body debug block to be like: \n\nif headers.get(\u0027Content-Type\u0027) in (\u0027application/json\u0027, \u0027text/html\u0027):\n    string_parts.append(data) \n\nand just not print the body for unknown content types. Printing decoded data blobs is not much use anyway.","accounts_in_message":[],"_revision_number":8},{"id":"7f4a60d0b4cd32d43dd1757a13072ae632c5a85b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-02-12 03:46:42.000000000","message":"Patch Set 8: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":8},{"id":"09b9c50f03bb540bf35a4f1f465fdd238a35620e","date":"2016-02-12 04:37:43.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":8},{"id":"fdf211bed07c8e0aacfcc2cc527b62cb51b77114","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-02-12 04:37:43.000000000","message":"Patch Set 8: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-keystoneclient-docs http://docs-draft.openstack.org/27/278027/8/gate/gate-python-keystoneclient-docs/abcd6f7//doc/build/html/ : SUCCESS in 1m 05s\n- gate-python-keystoneclient-pep8 http://logs.openstack.org/27/278027/8/gate/gate-python-keystoneclient-pep8/91e68a9/ : SUCCESS in 52s\n- gate-python-keystoneclient-python27 http://logs.openstack.org/27/278027/8/gate/gate-python-keystoneclient-python27/f316895/ : SUCCESS in 1m 29s\n- gate-python-keystoneclient-python34 http://logs.openstack.org/27/278027/8/gate/gate-python-keystoneclient-python34/720576b/ : SUCCESS in 3m 40s\n- gate-tempest-dsvm-neutron-src-python-keystoneclient http://logs.openstack.org/27/278027/8/gate/gate-tempest-dsvm-neutron-src-python-keystoneclient/6a3839c/ : SUCCESS in 50m 52s\n- gate-keystoneclient-dsvm-functional http://logs.openstack.org/27/278027/8/gate/gate-keystoneclient-dsvm-functional/4e4a437/ : SUCCESS in 13m 45s","accounts_in_message":[],"_revision_number":8}],"current_revision_number":8,"current_revision":"96fbfeab973cf61cd5ba129056cf4e6634d805b8","revisions":{"d0d14d54f165f1332babb2f26bf57c6ab36e80ae":{"kind":"REWORK","_number":1,"created":"2016-02-09 18:44:57.000000000","uploader":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"ref":"refs/changes/27/278027/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-keystoneclient","ref":"refs/changes/27/278027/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/1"}}},"commit":{"parents":[{"commit":"81f7fea507025229ba258d7ec7467867fe014438","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/81f7fea507025229ba258d7ec7467867fe014438"}]}],"author":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-09 18:40:39.000000000","tz":-300},"committer":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-09 18:44:49.000000000","tz":-300},"subject":"Handle exception on UnicodeDecodError in logging of request","message":"Handle exception on UnicodeDecodError in logging of request\n\nIf the logging of data in an HTTP request causes a UnicodeDecodeError,\nretry the the log entry using data.decode(\"ascii\", \"replace\") in\nplace of the original data.\n\nChange-Id: Ic365c654ebca4045208c6c30e232665145db7b4c\nCloses-Bug: #1453953\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/d0d14d54f165f1332babb2f26bf57c6ab36e80ae"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/d0d14d54f165f1332babb2f26bf57c6ab36e80ae"}]},"branch":"refs/heads/master"},"5af1d1ced61bcdcad96f6ead111186e721c4ac84":{"kind":"REWORK","_number":2,"created":"2016-02-09 19:28:26.000000000","uploader":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"ref":"refs/changes/27/278027/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-keystoneclient","ref":"refs/changes/27/278027/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/2"}}},"commit":{"parents":[{"commit":"81f7fea507025229ba258d7ec7467867fe014438","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/81f7fea507025229ba258d7ec7467867fe014438"}]}],"author":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-09 18:40:39.000000000","tz":-300},"committer":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-09 19:27:13.000000000","tz":-300},"subject":"Handle exception on UnicodeDecodError in logging of request","message":"Handle exception on UnicodeDecodError in logging of request\n\nIf the logging of an HTTP request causes a UnicodeDecodeError,\nretry the the log entry using message.decode(\"ascii\", \"replace\") in\nplace of the original message.\n\nChange-Id: Ic365c654ebca4045208c6c30e232665145db7b4c\nCloses-Bug: #1453953\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/5af1d1ced61bcdcad96f6ead111186e721c4ac84"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/5af1d1ced61bcdcad96f6ead111186e721c4ac84"}]},"branch":"refs/heads/master"},"39eb59e2b1b964a2b7a87346655745ed265ada6e":{"kind":"REWORK","_number":3,"created":"2016-02-09 20:23:27.000000000","uploader":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"ref":"refs/changes/27/278027/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-keystoneclient","ref":"refs/changes/27/278027/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/3"}}},"commit":{"parents":[{"commit":"81f7fea507025229ba258d7ec7467867fe014438","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/81f7fea507025229ba258d7ec7467867fe014438"}]}],"author":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-09 18:40:39.000000000","tz":-300},"committer":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-09 20:21:41.000000000","tz":-300},"subject":"Handle exception on UnicodeDecodError in logging of request","message":"Handle exception on UnicodeDecodError in logging of request\n\nIf the logging of an HTTP request causes a UnicodeDecodeError,\nretry the the log entry using message.decode(\"ascii\", \"replace\") in\nplace of the original message.\n\nCo-Authored-By: Nikita Konovalov \u003cnkonovalov@mirantis.com\u003e\n\nChange-Id: Ic365c654ebca4045208c6c30e232665145db7b4c\nCloses-Bug: #1453953\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/39eb59e2b1b964a2b7a87346655745ed265ada6e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/39eb59e2b1b964a2b7a87346655745ed265ada6e"}]},"branch":"refs/heads/master"},"836a38b5ea2c35269e74429ad8a341fb26647060":{"kind":"REWORK","_number":4,"created":"2016-02-09 20:30:27.000000000","uploader":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"ref":"refs/changes/27/278027/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-keystoneclient","ref":"refs/changes/27/278027/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/4"}}},"commit":{"parents":[{"commit":"81f7fea507025229ba258d7ec7467867fe014438","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/81f7fea507025229ba258d7ec7467867fe014438"}]}],"author":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-09 18:40:39.000000000","tz":-300},"committer":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-09 20:26:24.000000000","tz":0},"subject":"Handle exception on UnicodeDecodError in logging of request","message":"Handle exception on UnicodeDecodError in logging of request\n\nIf the logging of an HTTP request causes a UnicodeDecodeError,\nretry the the log entry using message.decode(\"ascii\", \"replace\") in\nplace of the original message.\n\nCo-Authored-By: Nikita Konovalov \u003cnkonovalov@mirantis.com\u003e\n\nChange-Id: Ic365c654ebca4045208c6c30e232665145db7b4c\nCloses-Bug: #1453953\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/836a38b5ea2c35269e74429ad8a341fb26647060"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/836a38b5ea2c35269e74429ad8a341fb26647060"}]},"branch":"refs/heads/master"},"bf22c5f948e46d0a27939e2b32a3067115841e03":{"kind":"REWORK","_number":5,"created":"2016-02-10 17:38:51.000000000","uploader":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"ref":"refs/changes/27/278027/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-keystoneclient","ref":"refs/changes/27/278027/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/5"}}},"commit":{"parents":[{"commit":"81f7fea507025229ba258d7ec7467867fe014438","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/81f7fea507025229ba258d7ec7467867fe014438"}]}],"author":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-09 18:40:39.000000000","tz":-300},"committer":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-10 17:35:53.000000000","tz":-300},"subject":"Handle exception on UnicodeDecodError in logging of request","message":"Handle exception on UnicodeDecodError in logging of request\n\nIf the logging of an HTTP request causes a UnicodeDecodeError,\nmodify the log entry using oslo_utils.encode_utils.safe_decode\nwith errors\u003d\u0027replace\u0027 and try again\n\nCo-Authored-By: Nikita Konovalov \u003cnkonovalov@mirantis.com\u003e\n\nChange-Id: Ic365c654ebca4045208c6c30e232665145db7b4c\nCloses-Bug: #1453953\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/bf22c5f948e46d0a27939e2b32a3067115841e03"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/bf22c5f948e46d0a27939e2b32a3067115841e03"}]},"branch":"refs/heads/master"},"8634cd61776dfa8b5d0a0ae728783e436e792806":{"kind":"REWORK","_number":6,"created":"2016-02-10 17:45:04.000000000","uploader":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"ref":"refs/changes/27/278027/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-keystoneclient","ref":"refs/changes/27/278027/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/6"}}},"commit":{"parents":[{"commit":"81f7fea507025229ba258d7ec7467867fe014438","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/81f7fea507025229ba258d7ec7467867fe014438"}]}],"author":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-09 18:40:39.000000000","tz":-300},"committer":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-10 17:44:55.000000000","tz":-300},"subject":"Handle exception on UnicodeDecodError in logging of request","message":"Handle exception on UnicodeDecodError in logging of request\n\nIf the logging of an HTTP request causes a UnicodeDecodeError,\nmodify the log entry using oslo_utils.encodeutils.safe_decode\nwith errors\u003d\u0027replace\u0027 and try again\n\nCo-Authored-By: Nikita Konovalov \u003cnkonovalov@mirantis.com\u003e\n\nChange-Id: Ic365c654ebca4045208c6c30e232665145db7b4c\nCloses-Bug: #1453953\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/8634cd61776dfa8b5d0a0ae728783e436e792806"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/8634cd61776dfa8b5d0a0ae728783e436e792806"}]},"branch":"refs/heads/master"},"6aa2bf99e4bf7a12f41ebbcaa378c215f1f3850f":{"kind":"REWORK","_number":7,"created":"2016-02-10 17:49:06.000000000","uploader":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"ref":"refs/changes/27/278027/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-keystoneclient","ref":"refs/changes/27/278027/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/7"}}},"commit":{"parents":[{"commit":"81f7fea507025229ba258d7ec7467867fe014438","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/81f7fea507025229ba258d7ec7467867fe014438"}]}],"author":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-09 18:40:39.000000000","tz":-300},"committer":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-10 17:49:00.000000000","tz":-300},"subject":"Handle exception on UnicodeDecodError in logging of request","message":"Handle exception on UnicodeDecodError in logging of request\n\nIf the logging of an HTTP request causes a UnicodeDecodeError,\nmodify the log entry using oslo_utils.encodeutils.safe_decode\nwith errors\u003d\u0027replace\u0027 and try again\n\nCo-Authored-By: Nikita Konovalov \u003cnkonovalov@mirantis.com\u003e\n\nChange-Id: Ic365c654ebca4045208c6c30e232665145db7b4c\nCloses-Bug: #1453953\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/6aa2bf99e4bf7a12f41ebbcaa378c215f1f3850f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/6aa2bf99e4bf7a12f41ebbcaa378c215f1f3850f"}]},"branch":"refs/heads/master"},"96fbfeab973cf61cd5ba129056cf4e6634d805b8":{"kind":"REWORK","_number":8,"created":"2016-02-10 19:04:56.000000000","uploader":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"ref":"refs/changes/27/278027/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-keystoneclient","ref":"refs/changes/27/278027/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-keystoneclient refs/changes/27/278027/8"}}},"commit":{"parents":[{"commit":"81f7fea507025229ba258d7ec7467867fe014438","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/81f7fea507025229ba258d7ec7467867fe014438"}]}],"author":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-09 18:40:39.000000000","tz":-300},"committer":{"name":"Trevor McKay","email":"tmckay@redhat.com","date":"2016-02-10 19:04:47.000000000","tz":-300},"subject":"Handle exception on UnicodeDecodError in logging of request","message":"Handle exception on UnicodeDecodError in logging of request\n\nIf the logging of an HTTP request causes a UnicodeDecodeError,\nmodify the log entry using oslo_utils.encodeutils.safe_decode\nwith errors\u003d\u0027replace\u0027 and try again\n\nCo-Authored-By: Nikita Konovalov \u003cnkonovalov@mirantis.com\u003e\n\nChange-Id: Ic365c654ebca4045208c6c30e232665145db7b4c\nCloses-Bug: #1453953\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/96fbfeab973cf61cd5ba129056cf4e6634d805b8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/96fbfeab973cf61cd5ba129056cf4e6634d805b8"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
