tobias/radicale
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use repr(...) instead of "%r" % ...

Browse Source
This commit is contained in:
Unrud 2017-07-22 23:20:51 +02:00 committed by Unrud
parent d0aacd3dcf
commit 1f4fb9e306
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
radicale/__init__.py
View File

@ -289,7 +289,7 @@ class Application:
write_allowed_items.append(item) write_allowed_items.append(item)
self.logger.debug( self.logger.debug(
"%s has %s access to %s", "%s has %s access to %s",
"%r" % user if user else "anonymous user", repr(user) if user else "anonymous user",
" and ".join(text_status) if text_status else "NO", target) " and ".join(text_status) if text_status else "NO", target)
return read_allowed_items, write_allowed_items return read_allowed_items, write_allowed_items
@ -355,7 +355,7 @@ class Application:
remote_host = "unknown" remote_host = "unknown"
if environ.get("REMOTE_HOST"): if environ.get("REMOTE_HOST"):
remote_host = "%r" % environ["REMOTE_HOST"] remote_host = repr(environ["REMOTE_HOST"])
elif environ.get("REMOTE_ADDR"): elif environ.get("REMOTE_ADDR"):
remote_host = environ["REMOTE_ADDR"] remote_host = environ["REMOTE_ADDR"]
if environ.get("HTTP_X_FORWARDED_FOR"): if environ.get("HTTP_X_FORWARDED_FOR"):
@ -464,7 +464,7 @@ class Application:
environ, base_prefix, path, user) environ, base_prefix, path, user)
if (status, headers, answer) == NOT_ALLOWED: if (status, headers, answer) == NOT_ALLOWED:
self.logger.info("Access to %r denied for %s", path, self.logger.info("Access to %r denied for %s", path,
"%r" % user if user else "anonymous user") repr(user) if user else "anonymous user")
else: else:
status, headers, answer = NOT_ALLOWED status, headers, answer = NOT_ALLOWED