From 5dc6f94ffd3bbf0d45559068c86da4247bd454e8 Mon Sep 17 00:00:00 2001 From: Valery Febvre Date: Sat, 21 Aug 2004 17:12:21 +0000 Subject: Moved param 'http_method' of lasso_login_build_authn_request_msg() method in lasso_login_init_authn_request() BEWARE : this change breaks the API --- python/lasso.py | 8 ++++---- python/tests/ServiceProvider.py | 5 +++-- 2 files changed, 7 insertions(+), 6 deletions(-) (limited to 'python') diff --git a/python/lasso.py b/python/lasso.py index 461e5f4a..ce8ede4f 100644 --- a/python/lasso.py +++ b/python/lasso.py @@ -472,9 +472,9 @@ class Login(_ObjectMixin, lassomod.LassoLogin, _ProfileChild): if errorCode: raise newError(errorCode, 'lasso_login_build_artifact_msg') - def build_authn_request_msg(self, remote_providerID, http_method): + def build_authn_request_msg(self, remote_providerID): errorCode = lassomod.lasso_login_build_authn_request_msg( - self, remote_providerID, http_method) + self, remote_providerID) if errorCode: raise newError(errorCode, 'lasso_login_build_authn_request_msg') @@ -493,8 +493,8 @@ class Login(_ObjectMixin, lassomod.LassoLogin, _ProfileChild): def dump(self): return lassomod.lasso_login_dump(self) - def init_authn_request(self): - errorCode = lassomod.lasso_login_init_authn_request(self) + def init_authn_request(self, http_method): + errorCode = lassomod.lasso_login_init_authn_request(self, http_method) if errorCode: raise newError(errorCode, 'lasso_login_init_authn_request') diff --git a/python/tests/ServiceProvider.py b/python/tests/ServiceProvider.py index fdf3b7ff..b906dec9 100644 --- a/python/tests/ServiceProvider.py +++ b/python/tests/ServiceProvider.py @@ -223,7 +223,8 @@ class ServiceProviderMixin(Provider.ProviderMixin): return handler.respond(headers = headers, body = authnRequestEnvelopeMsg) else: login = lasso.Login(lassoServer) - login.init_authn_request() + login.init_authn_request(lasso.httpMethodRedirect) + #login.init_authn_request() failUnlessEqual(login.request_type, lasso.messageTypeAuthnRequest) if forceAuthn: login.request.set_forceAuthn(forceAuthn) @@ -233,7 +234,7 @@ class ServiceProviderMixin(Provider.ProviderMixin): login.request.set_consent(lasso.libConsentObtained) if relayState: login.request.set_relayState(relayState) - login.build_authn_request_msg(self.idpSite.providerId, lasso.httpMethodRedirect) + login.build_authn_request_msg(self.idpSite.providerId) authnRequestUrl = login.msg_url failUnless(authnRequestUrl) return handler.respondRedirectTemporarily(authnRequestUrl) -- cgit