)]}'
{"id":"openstack%2Fkeystoneauth~310043","triplet_id":"openstack%2Fkeystoneauth~master~I5797f490f2e57d1c952e769bc0ef4b96c08f9a83","project":"openstack/keystoneauth","branch":"master","topic":"bug/1501918","hashtags":[],"change_id":"I5797f490f2e57d1c952e769bc0ef4b96c08f9a83","subject":"Add 303 as redirect code for k2k plugin","status":"MERGED","created":"2016-04-25 19:28:54.000000000","updated":"2016-04-28 07:54:22.000000000","submitted":"2016-04-28 07:54:22.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"eef1a3d07d0072dbe666953e370b56851ef46cec","_number":310043,"virtual_id_number":310043,"owner":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},{"value":0,"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":2,"date":"2016-04-28 07:54:22.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"all":[{"value":0,"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},{"value":2,"date":"2016-04-25 22:31:23.000000000","_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},{"value":2,"date":"2016-04-28 07:02:29.000000000","_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"all":[{"value":0,"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},{"value":0,"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},{"value":1,"date":"2016-04-28 07:02:29.000000000","_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},{"_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"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-04-25 19:30:42.000000000","updated_by":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"reviewer":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"state":"REVIEWER"},{"updated":"2016-04-25 22:31:23.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-04-28 07:02:29.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-04-28 07:54:22.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"64f090e48239b0a50e65b98564849d8be68192f1","author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"date":"2016-04-25 19:28:54.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"a90c620a1e55df4c76d1827ca466fa33e7be2dd0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-04-25 20:25:11.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-keystoneauth-docs http://docs-draft.openstack.org/43/310043/1/check/gate-keystoneauth-docs/31e4773//doc/build/html/ : SUCCESS in 5m 05s\n- gate-keystoneauth-pep8 http://logs.openstack.org/43/310043/1/check/gate-keystoneauth-pep8/5318187/ : SUCCESS in 2m 33s\n- gate-keystoneauth-python27 http://logs.openstack.org/43/310043/1/check/gate-keystoneauth-python27/182c12d/ : SUCCESS in 2m 04s\n- gate-keystoneauth-python34 http://logs.openstack.org/43/310043/1/check/gate-keystoneauth-python34/bccc138/ : SUCCESS in 4m 00s\n- gate-tempest-dsvm-neutron-src-keystoneauth http://logs.openstack.org/43/310043/1/check/gate-tempest-dsvm-neutron-src-keystoneauth/3bea53c/ : SUCCESS in 56m 10s","accounts_in_message":[],"_revision_number":1},{"id":"7fb2a6e61ce6679c3511244003935298c2e6e30a","author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"date":"2016-04-25 21:43:03.000000000","message":"Patch Set 1: Code-Review+2\n\nI\u0027m sure we\u0027ve done this patch a few times. \n\nIdeally this should include a test.","accounts_in_message":[],"_revision_number":1},{"id":"76bf0835af30367662003e89e9535b68417cc8d7","author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"date":"2016-04-25 21:54:55.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"1603538e38725e475f2bc818f6a7e40eaa3cc734","author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"date":"2016-04-25 21:55:19.000000000","message":"Patch Set 2:\n\n@Jamie, you are right, added a couple of tests for it.","accounts_in_message":[],"_revision_number":2},{"id":"c706694848863d3c55f9bc562bfe9e6f0ed86ba3","author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"date":"2016-04-25 22:31:23.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"1984397120ea5dc3ecfd377a33b6e5cf035c1f17","author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"date":"2016-04-25 22:31:37.000000000","message":"Patch Set 2:\n\nThanks for the test","accounts_in_message":[],"_revision_number":2},{"id":"a46c206f3f7b57a9544994f65159e536bb176721","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-04-25 22:57:39.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-keystoneauth-docs http://docs-draft.openstack.org/43/310043/2/check/gate-keystoneauth-docs/78b1b2e//doc/build/html/ : SUCCESS in 3m 05s\n- gate-keystoneauth-pep8 http://logs.openstack.org/43/310043/2/check/gate-keystoneauth-pep8/c3ead7c/ : SUCCESS in 4m 15s\n- gate-keystoneauth-python27 http://logs.openstack.org/43/310043/2/check/gate-keystoneauth-python27/7c65e9a/ : SUCCESS in 1m 24s\n- gate-keystoneauth-python34 http://logs.openstack.org/43/310043/2/check/gate-keystoneauth-python34/8af5b48/ : SUCCESS in 3m 53s\n- gate-tempest-dsvm-neutron-src-keystoneauth http://logs.openstack.org/43/310043/2/check/gate-tempest-dsvm-neutron-src-keystoneauth/427cb34/ : SUCCESS in 1h 02m 38s","accounts_in_message":[],"_revision_number":2},{"id":"c74cc7f00598c600495245d023d5d91038cf5b6b","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-04-28 07:02:29.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\nplease backport if necessary","accounts_in_message":[],"_revision_number":2},{"id":"d2ffa4197d99d9180889089edc3af0ebf3ddb607","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-04-28 07:02:34.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"cb307c72d6acabaed980116c679d5e2aabc32182","date":"2016-04-28 07:54:22.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":2},{"id":"50fdcd1b828374b8528bcb025e88ae91fdf3337a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-04-28 07:54:22.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-keystoneauth-docs http://docs-draft.openstack.org/43/310043/2/gate/gate-keystoneauth-docs/4b8d22e//doc/build/html/ : SUCCESS in 2m 54s\n- gate-keystoneauth-pep8 http://logs.openstack.org/43/310043/2/gate/gate-keystoneauth-pep8/de5cd0a/ : SUCCESS in 2m 46s\n- gate-keystoneauth-python27 http://logs.openstack.org/43/310043/2/gate/gate-keystoneauth-python27/41d36dc/ : SUCCESS in 2m 48s\n- gate-keystoneauth-python34 http://logs.openstack.org/43/310043/2/gate/gate-keystoneauth-python34/f6048eb/ : SUCCESS in 4m 03s\n- gate-tempest-dsvm-neutron-src-keystoneauth http://logs.openstack.org/43/310043/2/gate/gate-tempest-dsvm-neutron-src-keystoneauth/d1b9f1c/ : SUCCESS in 51m 47s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"1a2a579393a4b7fde168fe42c1b8e8a71660f382","revisions":{"3a17d589da5c781bd62d089afbf6b00cf4489f3b":{"kind":"REWORK","_number":1,"created":"2016-04-25 19:28:54.000000000","uploader":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"ref":"refs/changes/43/310043/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystoneauth","ref":"refs/changes/43/310043/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystoneauth refs/changes/43/310043/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystoneauth refs/changes/43/310043/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystoneauth refs/changes/43/310043/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystoneauth refs/changes/43/310043/1"}}},"commit":{"parents":[{"commit":"9d51a2ae6052dee46198ed3799365a6480cb42c4","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystoneauth/commit/9d51a2ae6052dee46198ed3799365a6480cb42c4"}]}],"author":{"name":"Rodrigo Duarte","email":"rduartes@redhat.com","date":"2016-04-25 19:19:48.000000000","tz":-180},"committer":{"name":"Rodrigo Duarte","email":"rduartes@redhat.com","date":"2016-04-25 19:28:54.000000000","tz":-180},"subject":"Add 303 as redirect code for k2k plugin","message":"Add 303 as redirect code for k2k plugin\n\nSome service providers, like mod_mellon return a 303 response\nupon successful authentication. The \"requests\" package only\nconsiders 303, as per the following example::\n\n  \u003e\u003e\u003e import requests\n  \u003e\u003e\u003e requests.codes[\u0027found\u0027]\n  302\n\nChange-Id: I5797f490f2e57d1c952e769bc0ef4b96c08f9a83\nRelated-Bug: 1501918\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystoneauth/commit/3a17d589da5c781bd62d089afbf6b00cf4489f3b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystoneauth/commit/3a17d589da5c781bd62d089afbf6b00cf4489f3b"}]},"branch":"refs/heads/master"},"1a2a579393a4b7fde168fe42c1b8e8a71660f382":{"kind":"REWORK","_number":2,"created":"2016-04-25 21:54:55.000000000","uploader":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"ref":"refs/changes/43/310043/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystoneauth","ref":"refs/changes/43/310043/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystoneauth refs/changes/43/310043/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystoneauth refs/changes/43/310043/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystoneauth refs/changes/43/310043/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystoneauth refs/changes/43/310043/2"}}},"commit":{"parents":[{"commit":"9d51a2ae6052dee46198ed3799365a6480cb42c4","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystoneauth/commit/9d51a2ae6052dee46198ed3799365a6480cb42c4"}]}],"author":{"name":"Rodrigo Duarte","email":"rduartes@redhat.com","date":"2016-04-25 19:19:48.000000000","tz":-180},"committer":{"name":"Rodrigo Duarte","email":"rduartes@redhat.com","date":"2016-04-25 21:53:19.000000000","tz":-180},"subject":"Add 303 as redirect code for k2k plugin","message":"Add 303 as redirect code for k2k plugin\n\nSome service providers, like mod_mellon return a 303 response\nupon successful authentication. The \"requests\" package only\nconsiders 302, as per the following example::\n\n  \u003e\u003e\u003e import requests\n  \u003e\u003e\u003e requests.codes[\u0027found\u0027]\n  302\n\nChange-Id: I5797f490f2e57d1c952e769bc0ef4b96c08f9a83\nRelated-Bug: 1501918\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystoneauth/commit/1a2a579393a4b7fde168fe42c1b8e8a71660f382"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystoneauth/commit/1a2a579393a4b7fde168fe42c1b8e8a71660f382"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
