Skip to content

Commit 41c8250

Browse files
author
Hans Hörberg
committed
Fix for the example repose sp.
Made it compatible with the API change: create_*_request method returns a tuple (request_id, request)
1 parent 09f19c7 commit 41c8250

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

.gitignore

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -169,3 +169,7 @@ example/sp-repoze/sp.xml
169169
example/sp-repoze/sp_conf.py
170170

171171
example/sp-repoze/sp_conf.py
172+
173+
example/idp2/sp_test.xml
174+
175+
example/sp-repoze/idp_test.xml

src/s2repoze/plugins/sp.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -383,9 +383,10 @@ def challenge(self, environ, _status, _app_headers, _forget_headers):
383383

384384
if _cli.authn_requests_signed:
385385
_sid = saml2.s_utils.sid(_cli.seed)
386-
msg_id = msg_str = _cli.create_authn_request(
386+
req_id, msg_str = _cli.create_authn_request(
387387
dest, vorg=vorg_name, sign=_cli.authn_requests_signed,
388388
message_id=_sid, extensions=extensions)
389+
req_id = _sid
389390
else:
390391
req_id, req = _cli.create_authn_request(
391392
dest, vorg=vorg_name, sign=False, extensions=extensions)

0 commit comments

Comments
 (0)