Skip to content

Commit a59e1db

Browse files
author
Rebecka Gulliksson
committed
Update consent micro service to work with new API.
1 parent bc035f7 commit a59e1db

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

src/satosa/micro_services/consent.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ def _approve_new_consent(self, context, internal_response, id_hash):
9999
internal_response.attributes = {}
100100
return self._end_consent(context, internal_response)
101101

102-
consent_redirect = "%s?ticket=%s" % (self.redirect_url, ticket)
102+
consent_redirect = "%s/%s" % (self.redirect_url, ticket)
103103
return Redirect(consent_redirect)
104104

105105
def process(self, context, internal_response):

tests/satosa/micro_services/test_consent.py

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import json
22
import re
33
from collections import Counter
4-
from unittest.mock import Mock
54
from urllib.parse import urlparse, parse_qs
65

76
import pytest
@@ -65,10 +64,8 @@ def consent_registration_endpoint_regex(self):
6564
def assert_redirect(self, redirect_resp, expected_ticket):
6665
assert isinstance(redirect_resp, Redirect)
6766

68-
parsed_url = parse_qs(urlparse(redirect_resp.message).query)
69-
assert len(parsed_url["ticket"]) == 1
70-
ticket = parsed_url["ticket"][0]
71-
assert ticket == expected_ticket
67+
path = urlparse(redirect_resp.message).path
68+
assert path == "/consent/" + expected_ticket
7269

7370
def assert_registration_req(self, request, internal_response, sign_key_path, base_url):
7471
split_path = request.path_url.lstrip("/").split("/")

0 commit comments

Comments
 (0)