diff options
| author | Simo Sorce <simo@redhat.com> | 2014-09-10 17:20:02 -0400 |
|---|---|---|
| committer | Patrick Uiterwijk <puiterwijk@redhat.com> | 2014-09-24 20:51:47 +0200 |
| commit | b4bcb99e3217e658c1277cd5d484fa0c62c7aa0c (patch) | |
| tree | 43ac5b08ed35f207eba1bc34dba89162d847a95f /ipsilon/providers/saml2idp.py | |
| parent | e0895efb26de64a28de7b9219f524b715c396b2b (diff) | |
| download | ipsilon-b4bcb99e3217e658c1277cd5d484fa0c62c7aa0c.tar.gz ipsilon-b4bcb99e3217e658c1277cd5d484fa0c62c7aa0c.tar.xz ipsilon-b4bcb99e3217e658c1277cd5d484fa0c62c7aa0c.zip | |
Use transactions throughout the code
Signed-off-by: Simo Sorce <simo@redhat.com>
Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
Diffstat (limited to 'ipsilon/providers/saml2idp.py')
| -rwxr-xr-x | ipsilon/providers/saml2idp.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/ipsilon/providers/saml2idp.py b/ipsilon/providers/saml2idp.py index a19899c..e30e4a1 100755 --- a/ipsilon/providers/saml2idp.py +++ b/ipsilon/providers/saml2idp.py @@ -60,8 +60,8 @@ class Continue(AuthenticateRequest): session = UserSession() user = session.get_user() - session.nuke_data('login', 'Return') - self.stage = session.get_data('saml2', 'stage') + transdata = self.trans.retrieve() + self.stage = transdata['saml2_stage'] if user.is_anonymous: self._debug("User is marked anonymous?!") @@ -70,11 +70,11 @@ class Continue(AuthenticateRequest): self._debug('Continue auth for %s' % user.name) - dump = session.get_data('saml2', 'Request') - if not dump: + if 'saml2_request' not in transdata: self._debug("Couldn't find Request dump?!") # TODO: Return to SP with auth failed error raise cherrypy.HTTPError(400) + dump = transdata['saml2_request'] try: login = self.cfg.idp.get_login_handler(dump) |