cyclic dependencies in implied roles

Bug #1536321 reported by Adam Young
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
High
Adam Young

Bug Description

Today it is possible to define an implied role structure that is not a DAG. This will crash the Keystone server if a token iis requested that will pull in any of those roles.

While it might be impractical to prevent cycles in the creation, it is very possible to prevent the expansion from crashing the server.

Changed in keystone:
importance: Undecided → High
milestone: none → mitaka-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

Fix proposed to branch: master
Review: https://review.openstack.org/271134

Changed in keystone:
assignee: nobody → Adam Young (ayoung)
status: New → In Progress
Changed in keystone:
assignee: Adam Young (ayoung) → Steve Martinelli (stevemar)
Changed in keystone:
assignee: Steve Martinelli (stevemar) → Adam Young (ayoung)
Changed in keystone:
assignee: Adam Young (ayoung) → Steve Martinelli (stevemar)
Changed in keystone:
assignee: Steve Martinelli (stevemar) → Adam Young (ayoung)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/271134
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=2a5599811e6164bf7e05f55d7157934a1ac7e3a3
Submitter: Jenkins
Branch: master

commit 2a5599811e6164bf7e05f55d7157934a1ac7e3a3
Author: Adam Young <email address hidden>
Date: Thu Jan 21 23:14:41 2016 -0500

    Check for circular references when expanding implied roles

    closes-bug #1536321

    Change-Id: I30d4f54135864d10093437a0541d6f40255e40e2

Changed in keystone:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.