We were using a one-liner to prune duplicate role references from a
list of roles, but it didn't work in all cases. This reworks the
logic to pass the existing test case. I also added a comment
explaining why the logic we used previously doesn't work so we can
hopefully avoid the pattern in the future.
Reviewed: https:/ /review. openstack. org/494049 /git.openstack. org/cgit/ openstack/ keystone/ commit/ ?id=058a23c0873 723d5a4ffa8e991 21f7b3b4485db5
Committed: https:/
Submitter: Jenkins
Branch: master
commit 058a23c0873723d 5a4ffa8e99121f7 b3b4485db5
Author: Lance Bragstad <email address hidden>
Date: Tue Aug 1 16:03:53 2017 +0000
Remove duplicate roles from federated auth
We were using a one-liner to prune duplicate role references from a
list of roles, but it didn't work in all cases. This reworks the
logic to pass the existing test case. I also added a comment
explaining why the logic we used previously doesn't work so we can
hopefully avoid the pattern in the future.
Change-Id: Id786d6463364ad 8f4f02c22bb8322 1baac4b83d0
Closes-Bug: 1701324