The "methods" modules conflicts with common usage in python

Bug #1136967 reported by Adam Young
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Wishlist
Dolph Mathews

Bug Description

IN the V3 Auth, we have put the links of the authenticaion chain into a directory called methods. WHile this may make sense to some, it is greatly confusing to programmers in general.

Possible alternative suggestions are:

Authenticators
Factors

Revision history for this message
termie (termie) wrote :

Strategies / Strategy
No directory at all (just leave them in the auth folder, why get needlessly architecty)

Revision history for this message
termie (termie) wrote :
Revision history for this message
Malini Bhandaru (malini-k-bhandaru) wrote :

Not factors. how about
"auth_methods" ?

Revision history for this message
Dolph Mathews (dolph) wrote :

The goal was to implement pluggable authentication, so why not call them "plugins"?

Changed in keystone:
importance: Undecided → Medium
status: New → Confirmed
milestone: none → grizzly-rc1
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/23721

Changed in keystone:
assignee: nobody → Dolph Mathews (dolph)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/23721
Committed: http://github.com/openstack/keystone/commit/ddd221630af049b577479d7f7ec0da8755b2a517
Submitter: Jenkins
Branch: master

commit ddd221630af049b577479d7f7ec0da8755b2a517
Author: Dolph Mathews <email address hidden>
Date: Wed Mar 6 12:53:47 2013 -0600

    Move auth plugins to 'keystone.auth.plugins' (bug 1136967)

    Change-Id: I8c72ee99695b0c039a91f807a13a832ce2c3ff74

Changed in keystone:
status: In Progress → Fix Committed
Dolph Mathews (dolph)
Changed in keystone:
importance: Medium → Wishlist
Thierry Carrez (ttx)
Changed in keystone:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: grizzly-rc1 → 2013.1
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.