ipsilon

Clone
Source Code
GIT

#44 Don't crash if no NameIdPolicy is requested
Merged 8 years ago by puiterwijk. Opened 8 years ago by rcritten.
rcritten/ipsilon nameidpolicy  into  master

file modified
+4 -2 
@@ -254,10 +254,12 @@ 
 

          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:
 

+             self.debug('Requested NameId [%s]' % (nip.format,))
 

              allowed = self.allowed_nameids
 

              self.debug('Allowed NameIds %s' % (repr(allowed)))
 

              for nameid in allowed:

no initial comment

Looks good to me.
Metadata
None
No Tags
Changes Summary 1
+4 -2
file changed
ipsilon/providers/saml2/provider.py
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.