openstack/python-keystoneclient
Code Issues Proposed changes

Python3: webob.Response.body must be bytes

Browse Source 
Remove duplicate assignement for 'resp.body'. Use bytes in Python3.

Change-Id: Ice1beb997ae401ebb87762e5499970409c63d801
This commit is contained in:
Cyril Roelandt
2014-02-06 17:11:39 +01:00
parent 3d0eacddf9
commit 9a2844e2d9
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
keystoneclient/middleware/s3_token.py
View File

@@ -38,6 +38,7 @@ import urllib
import webob import webob
import requests import requests
import six
from keystoneclient.openstack.common import jsonutils from keystoneclient.openstack.common import jsonutils
@@ -134,11 +135,13 @@ class S3Token(object):
} }
resp = webob.Response(content_type='text/xml') resp = webob.Response(content_type='text/xml')
resp.status = error_table[code][0] resp.status = error_table[code][0]
resp.body = error_table[code][1] error_msg = ('<?xml version="1.0" encoding="UTF-8"?>\r\n'
resp.body = ('<?xml version="1.0" encoding="UTF-8"?>\r\n'
'<Error>\r\n <Code>%s</Code>\r\n ' '<Error>\r\n <Code>%s</Code>\r\n '
'<Message>%s</Message>\r\n</Error>\r\n' % '<Message>%s</Message>\r\n</Error>\r\n' %
(code, error_table[code][1])) (code, error_table[code][1]))
if six.PY3:
error_msg = error_msg.encode()
resp.body = error_msg
return resp return resp
def _json_request(self, creds_json): def _json_request(self, creds_json):