From: Patrick Uiterwijk Date: Wed, 8 Jul 2015 10:37:04 +0000 (+0200) Subject: Only initialize the OpenID IDP when actually enabled X-Git-Tag: v1.1.0~58 X-Git-Url: http://git.cascardo.info/?p=cascardo%2Fipsilon.git;a=commitdiff_plain;h=ac7c20cca81c3d23ee66f224030b316bdff2274a Only initialize the OpenID IDP when actually enabled This is needed because otherwise it will try to set the database schema version before it read the configuration for providers, which means it will do this in the default (openid.sqlite) database file. If you are running as a non-privileged user (as your should) with the working directory pointing somewhere this user is unable to write, this means it will fail to write this. Note: the working directory is not in the default wsgi file, which means that people using that will not likely hit this bug. Signed-off-by: Patrick Uiterwijk Reviewed-by: Simo Sorce --- diff --git a/ipsilon/providers/openidp.py b/ipsilon/providers/openidp.py index 8f74578..dc47faf 100644 --- a/ipsilon/providers/openidp.py +++ b/ipsilon/providers/openidp.py @@ -104,7 +104,6 @@ Provides OpenID 2.0 authentication infrastructure. """ return self.get_config_value('default allowed attributes') def get_tree(self, site): - self.init_idp() self.page = OpenID(site, self) # self.admin = AdminPage(site, self)