diff options
| author | Patrick Uiterwijk <puiterwijk@redhat.com> | 2015-07-08 12:37:04 +0200 |
|---|---|---|
| committer | Patrick Uiterwijk <puiterwijk@redhat.com> | 2015-07-08 13:27:02 +0200 |
| commit | ac7c20cca81c3d23ee66f224030b316bdff2274a (patch) | |
| tree | 3f312b6949226e083ecb74ec6dfde9ee1f6bbfb1 | |
| parent | dc44adc21b7e8a2aac61b5a641bacc7d706608e1 (diff) | |
| download | ipsilon-ac7c20cca81c3d23ee66f224030b316bdff2274a.tar.gz ipsilon-ac7c20cca81c3d23ee66f224030b316bdff2274a.tar.xz ipsilon-ac7c20cca81c3d23ee66f224030b316bdff2274a.zip | |
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 <puiterwijk@redhat.com>
Reviewed-by: Simo Sorce <simo@redhat.com>
| -rw-r--r-- | ipsilon/providers/openidp.py | 1 |
1 files changed, 0 insertions, 1 deletions
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) |