openstack/deb-python-pysaml2
Code Issues Proposed changes

shortened "name" values; added "desrc" keys in some cases (used to improve GUI in STHREP)

Browse Source
This commit is contained in:
rhoerbe
2014-07-30 16:10:58 +02:00
parent 707a941252
commit 5ee6ebc897
1 changed files with 13 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/sp_test/tests.py
View File

@@ -11,7 +11,8 @@ from saml2.saml import SCM_SENDER_VOUCHES
from saml2.saml import ConditionAbstractType_ from saml2.saml import ConditionAbstractType_
from saml2.samlp import STATUS_AUTHN_FAILED from saml2.samlp import STATUS_AUTHN_FAILED
from saml2.time_util import in_a_while, a_while_ago from saml2.time_util import in_a_while, a_while_ago
from sp_test.check import VerifyAuthnRequest from sp_test.check import VerifyAuthnRequest, VerifyDigestAlgorithm, \
VerifySignatureAlgorithm, VerifyIfRequestIsSigned
from sp_test import check from sp_test import check
from saml2test import ip_addresses from saml2test import ip_addresses
@@ -62,7 +63,7 @@ class Response(object):
_args = {} _args = {}
_class = samlp.Response _class = samlp.Response
_sign = False _sign = False
tests = {"post": [], "pre": []} tests = {"pre": [], "post": []}
def __init__(self, conv): def __init__(self, conv):
self.args = self._args.copy() self.args = self._args.copy()
@@ -81,7 +82,10 @@ class Response(object):
class Request(object): class Request(object):
response = "" response = ""
_class = None _class = None
tests = {"post": [VerifyAuthnRequest], "pre": []} tests = {"pre": [],
"post": [VerifyAuthnRequest,
VerifyDigestAlgorithm,
VerifySignatureAlgorithm,]}
def __init__(self): def __init__(self):
pass pass
@@ -377,6 +381,12 @@ OPERATIONS = {
"sequence": [(Login, AuthnRequest, AuthnResponse, check.VerifyEchopageContents)], "sequence": [(Login, AuthnRequest, AuthnResponse, check.VerifyEchopageContents)],
"tests": {"pre": [], "post": []} "tests": {"pre": [], "post": []}
}, },
'sp-02': {
"name": 'Require AuthnRequest to be signed',
"descr": 'Same as SP-00, and check if a request signature can be found',
"sequence": [(Login, AuthnRequest, AuthnResponse, None)],
"tests": {"pre": [], "post": [VerifyIfRequestIsSigned]}
},
'sp-08': { 'sp-08': {
"name": "SP should accept a Response without a " "name": "SP should accept a Response without a "
"SubjectConfirmationData element. If confirmation method" "SubjectConfirmationData element. If confirmation method"