diff options
author | Brant Knudson <bknudson@us.ibm.com> | 2015-07-24 15:52:57 -0500 |
---|---|---|
committer | Brant Knudson <bknudson@us.ibm.com> | 2015-08-04 09:56:44 -0500 |
commit | 0c2fef51d2b90df088d30e9b6c5079caae7c6578 (patch) | |
tree | 349ba613dd991b3d9a8be1e74476e7e9be316e73 /keystoneclient/generic | |
parent | ada04acf4d60eafc0e31e6ac079ddb2c4f12e728 (diff) | |
download | python-keystoneclient-0c2fef51d2b90df088d30e9b6c5079caae7c6578.tar.gz |
Proper deprecation for HTTPClient.request methods
HTTPClient.request and related methods weren't properly
deprecated since they were only mentioned in the docstrings.
Proper deprecation requires use of warnings/debtcollector and
documentation.
Also, fixed places where the deprecated request method was called.
bp deprecations
Change-Id: I0a16933252937ca046793bb6eb2e5cc5da03c9ae
Diffstat (limited to 'keystoneclient/generic')
-rw-r--r-- | keystoneclient/generic/client.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/keystoneclient/generic/client.py b/keystoneclient/generic/client.py index 7ca39fb..f61edec 100644 --- a/keystoneclient/generic/client.py +++ b/keystoneclient/generic/client.py @@ -84,9 +84,9 @@ class Client(httpclient.HTTPClient): def _check_keystone_versions(self, url): """Calls Keystone URL and detects the available API versions.""" try: - resp, body = self.request(url, "GET", - headers={'Accept': - 'application/json'}) + resp, body = self._request(url, "GET", + headers={'Accept': + 'application/json'}) # Multiple Choices status code is returned by the root # identity endpoint, with references to one or more # Identity API versions -- v3 spec @@ -148,9 +148,9 @@ class Client(httpclient.HTTPClient): try: if not url.endswith("/"): url += '/' - resp, body = self.request("%sextensions" % url, "GET", - headers={'Accept': - 'application/json'}) + resp, body = self._request("%sextensions" % url, "GET", + headers={'Accept': + 'application/json'}) if resp.status_code in (200, 204): # some cases we get No Content if 'extensions' in body and 'values' in body['extensions']: # Parse correct format (per contract) |