Small pep8 related typo fixes

This commit is contained in:
Guillaume Ayoub 2011-07-26 09:02:21 +02:00
parent 8f488eb6bc
commit cdae6f04e9
5 changed files with 11 additions and 10 deletions

View File

@ -22,7 +22,7 @@
# This file is just a script, allow [a-z0-9]* variable names # This file is just a script, allow [a-z0-9]* variable names
# pylint: disable-msg=C0103 # pylint: disable-msg=C0103
# ``import radicale`` refers to the ``radicale`` module, not ``radicale.py`` # ``import radicale`` refers to the ``radicale`` module, not ``radicale.py``
# pylint: disable-msg=W0406 # pylint: disable-msg=W0406
""" """
@ -163,7 +163,7 @@ finally:
radicale.log.LOGGER.info("Stopping Radicale") radicale.log.LOGGER.info("Stopping Radicale")
for server in servers: for server in servers:
radicale.log.LOGGER.debug( radicale.log.LOGGER.debug(
"Closing server listening to %s port %s" % ( "Closing server listening to %s port %s" % (
server.server_name, server.server_port)) server.server_name, server.server_port))

View File

@ -110,7 +110,7 @@ class Application(object):
# pylint: disable=E0202 # pylint: disable=E0202
@staticmethod @staticmethod
def headers_log(environ): def headers_log(environ):
"""Remove environment variables from the headers for logging purpose.""" """Remove environment variables from the headers for logging."""
request_environ = dict(environ) request_environ = dict(environ)
for shell_variable in os.environ: for shell_variable in os.environ:
del request_environ[shell_variable] del request_environ[shell_variable]
@ -315,7 +315,8 @@ class Application(object):
def move(self, environ, calendars, content, user): def move(self, environ, calendars, content, user):
"""Manage MOVE request.""" """Manage MOVE request."""
from_calendar = calendars[0] from_calendar = calendars[0]
from_name = xmlutils.name_from_path(environ["PATH_INFO"], from_calendar) from_name = xmlutils.name_from_path(
environ["PATH_INFO"], from_calendar)
if from_name: if from_name:
item = from_calendar.get_item(from_name) item = from_calendar.get_item(from_name)
if item: if item:
@ -337,7 +338,6 @@ class Application(object):
# Moving calendars, not supported # Moving calendars, not supported
return client.FORBIDDEN, {}, None return client.FORBIDDEN, {}, None
def options(self, environ, calendars, content, user): def options(self, environ, calendars, content, user):
"""Manage OPTIONS request.""" """Manage OPTIONS request."""
headers = { headers = {
@ -386,7 +386,7 @@ class Application(object):
def report(self, environ, calendars, content, user): def report(self, environ, calendars, content, user):
"""Manage REPORT request.""" """Manage REPORT request."""
# TODO: support multiple calendars here # TODO: support multiple calendars here
calendar = calendars[0] calendar = calendars[0]
headers = {'Content-Type': 'text/xml'} headers = {'Content-Type': 'text/xml'}
answer = xmlutils.report(environ["PATH_INFO"], content, calendar) answer = xmlutils.report(environ["PATH_INFO"], content, calendar)

View File

@ -47,7 +47,8 @@ def has_right(owner, user, password):
CONNEXION.simple_bind_s(BINDDN, PASSWORD) CONNEXION.simple_bind_s(BINDDN, PASSWORD)
distinguished_name = "%s=%s" % (ATTRIBUTE, ldap.dn.escape_dn_chars(user)) distinguished_name = "%s=%s" % (ATTRIBUTE, ldap.dn.escape_dn_chars(user))
log.LOGGER.debug("LDAP bind for %s in base %s" % (distinguished_name, BASE)) log.LOGGER.debug(
"LDAP bind for %s in base %s" % (distinguished_name, BASE))
users = CONNEXION.search_s(BASE, ldap.SCOPE_ONELEVEL, distinguished_name) users = CONNEXION.search_s(BASE, ldap.SCOPE_ONELEVEL, distinguished_name)
if users: if users:
@ -61,6 +62,6 @@ def has_right(owner, user, password):
return True return True
else: else:
log.LOGGER.debug("User %s not found" % user) log.LOGGER.debug("User %s not found" % user)
log.LOGGER.debug("LDAP bind failed") log.LOGGER.debug("LDAP bind failed")
return False return False

View File

@ -53,7 +53,7 @@ def _sha1(hash_value, password):
"""Check if ``hash_value`` and ``password`` match using sha1 method.""" """Check if ``hash_value`` and ``password`` match using sha1 method."""
hash_value = hash_value.replace("{SHA}", "").encode("ascii") hash_value = hash_value.replace("{SHA}", "").encode("ascii")
password = password.encode(config.get("htpasswd_encoding", "stock")) password = password.encode(config.get("htpasswd_encoding", "stock"))
sha1 = hashlib.sha1() # pylint: disable=E1101 sha1 = hashlib.sha1() # pylint: disable=E1101
sha1.update(password) sha1.update(password)
return sha1.digest() == base64.b64decode(hash_value) return sha1.digest() == base64.b64decode(hash_value)

View File

@ -35,9 +35,9 @@ from radicale import config
LOGGER = logging.getLogger() LOGGER = logging.getLogger()
FILENAME = os.path.expanduser(config.get("logging", "config")) FILENAME = os.path.expanduser(config.get("logging", "config"))
def start(): def start():
"""Start the logging according to the configuration.""" """Start the logging according to the configuration."""
if os.path.exists(FILENAME): if os.path.exists(FILENAME):
# Configuration taken from file # Configuration taken from file
logging.config.fileConfig(FILENAME) logging.config.fileConfig(FILENAME)