Invalid token provider
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Identity (keystone) |
Won't Fix
|
Wishlist
|
Dave Chen |
Bug Description
When specify token provoider as "pkiz" instead of full class path of token provider as "keystone.
2014-12-14 04:48:18.710721 Traceback (most recent call last):
2014-12-14 04:48:18.710732 File "/var/www/
2014-12-14 04:48:18.710798 drivers = backends.
2014-12-14 04:48:18.710817 File "/opt/stack/
2014-12-14 04:48:18.710851 token_provider_
2014-12-14 04:48:18.710867 File "/opt/stack/
2014-12-14 04:48:18.710944 self.__
2014-12-14 04:48:18.710950 File "/opt/stack/
2014-12-14 04:48:18.710969 init(self, *args, **kwargs)
2014-12-14 04:48:18.710974 File "/opt/stack/
2014-12-14 04:48:18.711100 super(Manager, self)._
2014-12-14 04:48:18.711106 File "/opt/stack/
2014-12-14 04:48:18.711150 self.driver = importutils.
2014-12-14 04:48:18.711165 File "/usr/local/
2014-12-14 04:48:18.711226 return import_
2014-12-14 04:48:18.711244 File "/usr/local/
2014-12-14 04:48:18.711252 __import__(mod_str)
2014-12-14 04:48:18.711264 ValueError: Empty module name
I understand the comment in "keystone.conf" has show us how to config the provider but in my opinion, it should also support "pkiz|pki|uuid" besides the full class path, or at least, there should be one dedicated exception to show us what's wrong with the config. Seen from the code there is no invalid provider checking, so invalid token provider exception to show us what's wrong with the config is desired.
Changed in keystone: | |
assignee: | nobody → Dave Chen (wei-d-chen) |
description: | updated |
description: | updated |
This is a problem that we've had with *all* of our drivers loading. This is not exclusive to the token provider. The dedicated exception is a good idea, we should move away from the full-class path and move to stevedore longer term.