Skip to content
This repository has been archived by the owner on Sep 23, 2020. It is now read-only.

Commit

Permalink
problem with N>1 authorized kernels and spaces in list
Browse files Browse the repository at this point in the history
  • Loading branch information
timf committed Apr 29, 2010
1 parent 40bfeb9 commit b8ed501
Showing 1 changed file with 2 additions and 0 deletions.
@@ -1,4 +1,5 @@
import os
import string
import sys
import zope.interface

Expand Down Expand Up @@ -41,6 +42,7 @@ def validate(self):
self.authz_kernels = []
ak_list = authz_conf.split(",")
for name in ak_list:
name = string.strip(name)
if os.path.isabs(name):
raise InvalidConfig("cannot accept absolute path in kernels->authz_kernels configuration: '%s'" % name)
authzk = AuthzKernel()
Expand Down

1 comment on commit b8ed501

@jgehrcke
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Huhu :)

Just wanted to mention that the string functions in the string module are deprecated: http://docs.python.org/library/string.html#string-functions

import string is not necessary, name.strip() is sufficient.

Please sign in to comment.