projects
/
cascardo
/
ipsilon.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Don't crash if no NameIdPolicy is requested
[cascardo/ipsilon.git]
/
ipsilon
/
providers
/
saml2
/
provider.py
diff --git
a/ipsilon/providers/saml2/provider.py
b/ipsilon/providers/saml2/provider.py
index
6cbf5ab
..
6d46ad2
100644
(file)
--- a/
ipsilon/providers/saml2/provider.py
+++ b/
ipsilon/providers/saml2/provider.py
@@
-254,10
+254,12
@@
class ServiceProvider(ServiceProviderConfig):
self.load_config()
def get_valid_nameid(self, nip):
self.load_config()
def get_valid_nameid(self, nip):
- self.debug('Requested NameId [%s]' % (nip.format,))
- if nip.format is None:
+ if nip is None or nip.format is None:
+ self.debug('No NameId requested, returning default [%s]'
+ % SAML2_NAMEID_MAP[self.default_nameid])
return SAML2_NAMEID_MAP[self.default_nameid]
else:
return SAML2_NAMEID_MAP[self.default_nameid]
else:
+ self.debug('Requested NameId [%s]' % (nip.format,))
allowed = self.allowed_nameids
self.debug('Allowed NameIds %s' % (repr(allowed)))
for nameid in allowed:
allowed = self.allowed_nameids
self.debug('Allowed NameIds %s' % (repr(allowed)))
for nameid in allowed: