summaryrefslogtreecommitdiff
path: root/keystoneclient/auth/identity/v3/base.py
Commit message (Collapse)AuthorAgeFilesLines
* Update json module to jsonutilscao.yuan2019-02-251-2/+2
| | | | | | | | | | oslo project provide jsonutils, and keystoneclient use it in many place[1], this PS to update the remained json module to oslo jsonutils for consistency. [1]: https://github.com/openstack/python-keystoneclient/search?utf8=%E2%9C%93&q=jsonutils&type= Change-Id: Id5275b5e6b5bf8f6d54406dac7ab95a30828cf58
* Remove use of positional decoratorMorgan Fainberg2017-08-071-2/+0
| | | | | | | | | The positional decorator results in poorly maintainable code in a misguided effort to emulate python3's key-word-arg only notation and functionality. This patch removes keystoneclient's dependance on the positional decorator. Change-Id: I9e691cc8b0c04992f4a8dabd67e1b413d3220d23
* Fix 12 warnings when building keystoneclient docsGage Hugo2017-02-091-3/+3
| | | | | | | | | | | | | | | While building keystoneclient docs, there are currently 12 warnings emitted that specify either: WARNING: more than one target found for cross-reference u'list' WARNING: more than one target found for cross-reference u'Auth' This change specifies the correct object for the docstring with "List" since there are many instances of "list" within keystoneclient and specifies the proper "Auth" object. With these changes, the warnings no longer appear. Change-Id: I4515429df38760700552d48fc570c03abf116f83
* Allow seeing full token response when debug enabledStuart McLaren2016-03-151-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | In many situations it can be useful to see the full token response. Print v3 token response when in debug mode. This will allow the reponse to be seen when '--debug' is specified with the openstack CLI: { "token": { "methods": [ "password" ], "roles": [ { "id": "93bff41cabda4def87cc9d83aaaa7479" "name": "Member" }, . . . Change-Id: I9ec39ceed122a79bbaaef429750e1d2e8401297d Closes-bug: 1556977
* use positional library instead of utilsSteve Martinelli2016-01-251-2/+2
| | | | | | swap instances of utils.positional with the positional library. Change-Id: Id8a9961e68d287a802f25512fc970829e9feb5c2
* Address hacking check H405lin-hua-cheng2016-01-131-2/+4
| | | | | | | | | Previously, there were a string of commits to keystone that addresed ignored hacking checks. This commit does the same for H405 in keystoneclient. This also modifies our tox.ini so that we no longer ignore H405 violations. Change-Id: I2af152e5425a0e9c82314039fdbb90d661c22680 Closes-Bug: 1482773
* Mark abstractmethod bodies with nocoverBrant Knudson2015-10-211-2/+2
| | | | | | | | | abstractmethod bodies aren't going to be called by unit tests, so there's no way to get coverage. The code in an abstractmethod body should be marked with "# pragma: no cover" so that they don't show up as missed in the coverage report. Change-Id: I88a7481ab22f2ce1abfd62badc5f5048acc6929f
* pass on @abc.abstractmethodsDolph Mathews2015-10-211-0/+1
| | | | | | | | | | | | | | Although a bare docstring is entirely valid grammar, those new to Python find that pattern baffling, *especially* when other similar methods contain pass statements. So, for consistency, add a pass statement to otherwise bare @abc.abstractmethods. Note that the implementation of an @abc.abstractmethod (in the abstract base class) can still be called by concrete children, so suddenly raising a NotImplementedError() instead might be "surprising" to implementors. A no-op such as "pass" or "return None" is preferable. Change-Id: I79969ad1a3429516ea785c649a165ead54944225
* Make __all__ immutableDolph Mathews2015-10-011-1/+1
| | | | | | | | | Using a mutable type implies that it's acceptable for the set of publicly-accessible attributes to be mutated at runtime, which defeats their intended purpose of documenting the public interface. Tuples are immutable. Change-Id: Ib3ab93224ba240040b08ece481ef5ba620c3f658
* Stop using .keys() on dicts where not neededBrant Knudson2015-08-061-1/+1
| | | | | | | | Iterating over a dict results in the keys. Using the 'in' operator on a dict checks if it's a key. Change-Id: I6affbfa1a79a9e8c0b5b304078a7a8e4e792eecd
* Proper deprecation for BaseIdentityPlugin trust_id propertyBrant Knudson2015-07-261-1/+11
| | | | | | | | | | | | | BaseIdentityPlugin's trust_id property wasn't properly deprecated since all it had was a comment in the code. Proper deprecation requires use of warnings and documentation. Where the plugins already provide their own trust_id, the property needs to be un-deprecated. bp deprecations Change-Id: I15d4e019bfc5542990120ba39be65ad83cf040d5
* Allow requesting an unscoped TokenJamie Lennox2015-03-311-2/+9
| | | | | | | | | | | | The keystone server understands that specifying unscoped in the scope section of an auth request means that it should ignore the default_project_id of a user and return an unscoped token. This is the client side change to allow requesting these tokens via an auth plugin. Change-Id: Iba5ebcea0bf0d8e5a31d552977276fc03e536c67 Implements: bp explicit-unscoped
* Extract BaseAuth out of Auth PluginJamie Lennox2015-03-171-25/+51
| | | | | | | | | | | | | | | The basic Auth plugin for v3 tokens makes the assumption that you need to pass in some AuthMethod objects. This works well for most auth types where you want the plugin to construct the auth request for you. In the case of federation though we want to be able to have a rescoping plugin that will return an auth_ref and not take any auth_methods as arguments. Extract the most basic part of the Auth plugin into BaseAuth class that Auth and federation plugins can both inherit from. Change-Id: Ia8c8c614b8eb51170346ff5b1e20a1e7ebbb47de
* Split v3 authentication file into moduleJamie Lennox2015-03-171-0/+217
The V3 authentication plugins file contained the existing plugins as well as the base class. As we look to add new plugins it is simpler if we break this file up. Change-Id: I42b222a2012ea10491450d6b91c2008178dc7671