diff options
| author | Jay Faulkner <jay@jvf.cc> | 2020-01-07 11:50:43 -0800 |
|---|---|---|
| committer | Ruby Loo <opensrloo@gmail.com> | 2020-02-03 14:44:51 +0000 |
| commit | a09e82470fd934e1ecb46b53a1c0836ef4df744b (patch) | |
| tree | ce863ef83270e60a665c24e9e34e77b5a246b35e | |
| parent | 33ae5ad4e17fa4b38b5b7b0892d4ea8de2532361 (diff) | |
| download | python-neutronclient-stable/train.tar.gz | |
Convert exception to string before passing it intrain-em6.14.1stable/train
Before this change, neutronclient was passing in a raw exception
as a kwarg to the ConnectionFailed exception. This caused an
exception to be raised in _safe_decode_dict() due to the exception
not being a text type.
Now, we explicitly convert the raw exception to a string before
passing it as a kwarg.
Closes-bug: 1859068
Change-Id: I323b3aceec0a937874eabf770fbc82995202f6d6
(cherry picked from commit 946ac3ed2e9e177eb5c56cc74aadbc091b9292ab)
| -rw-r--r-- | neutronclient/client.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/neutronclient/client.py b/neutronclient/client.py index c5f4676..936d297 100644 --- a/neutronclient/client.py +++ b/neutronclient/client.py @@ -104,13 +104,13 @@ class HTTPClient(object): try: resp, body = self.request(*args, **kargs) except requests.exceptions.SSLError as e: - raise exceptions.SslCertificateValidationError(reason=e) + raise exceptions.SslCertificateValidationError(reason=str(e)) except Exception as e: # Wrap the low-level connection error (socket timeout, redirect # limit, decompression error, etc) into our custom high-level # connection exception (it is excepted in the upper layers of code) _logger.debug("throwing ConnectionFailed : %s", e) - raise exceptions.ConnectionFailed(reason=e) + raise exceptions.ConnectionFailed(reason=str(e)) utils.http_log_resp(_logger, resp, body) # log request-id for each api call |