Changeset 4119


Ignore:
Timestamp:
11/08/08 10:05:29 (11 years ago)
Author:
pjkersha
Message:

Fix to getSessionStatus for userDN input - X500DN instead of X509DN

File:
1 edited

Legend:

Unmodified
Added
Removed
  • TI12-security/trunk/python/ndg.security.server/ndg/security/server/SessionMgr/__init__.py

    r3955 r4119  
    162162            return dtNotAfter.strftime(self.__sessCookieExpiryFmt) 
    163163        except Exception, e: 
    164             UserSessionError, "getExpiry: %s" % e 
     164            UserSessionError("getExpiry: %s" % e) 
    165165 
    166166 
     
    188188                paddedURI = txt + ''.join(' '*nPad) 
    189189            except Exception, e: 
    190                 raise UserSessionError, "Padding text for encryption: %s" % e 
     190                raise UserSessionError("Padding text for encryption: %s" % e) 
    191191         
    192192            # Encrypt 
     
    195195             
    196196            except Exception, e: 
    197                 raise UserSessionError, "Encrypting Session Manager URI: %s"%e 
     197                raise UserSessionError("Encrypting Session Manager URI: %s"%e) 
    198198 
    199199        try: 
     
    201201         
    202202        except Exception, e: 
    203             raise UserSessionError, "Encoding Session Manager URI: %s"%e 
     203            raise UserSessionError("Encoding Session Manager URI: %s"%e) 
    204204         
    205205     
     
    228228             
    229229        except Exception, e: 
    230             raise SessionMgrError, "Decoding Session Manager URI: %s" % e            
     230            raise SessionMgrError("Decoding Session Manager URI: %s" % e)           
    231231 
    232232 
     
    239239             
    240240            except Exception, e: 
    241                 raise SessionMgrError, "Decrypting Session Manager URI: %s"%e            
     241                raise SessionMgrError("Decrypting Session Manager URI: %s"%e)            
    242242        else: 
    243243            return b64DecodedEncrTxt 
     
    282282             
    283283        elif not isinstance(sessID, basestring): 
    284             raise UserSessionError, "Input session ID is not a valid string" 
     284            raise UserSessionError("Input session ID is not a valid string") 
    285285                                 
    286286            if sessID not in self.__sessIDlist: 
    287                 raise UserSessionError, "Input session ID not found in list" 
     287                raise UserSessionError("Input session ID not found in list") 
    288288  
    289289  
     
    401401             
    402402        except Exception, e: 
    403             raise SessionMgrError, "Creating MyProxy interface: %s" % e 
     403            raise SessionMgrError("Creating MyProxy interface: %s" % e) 
    404404     
    405405        # Dictionary to hold properties       
     
    486486                 
    487487        except KeyError, e: 
    488             raise SessionMgrError, \ 
    489         'Missing %s element for credential repository module import' % str(e) 
     488            raise SessionMgrError('Missing %s element for credential ' 
     489                                  'repository module import' % str(e)) 
    490490                         
    491491        except Exception, e: 
    492             raise SessionMgrError, \ 
    493                         'Importing credential repository module: %s' % str(e) 
     492            raise SessionMgrError('Importing credential repository module: %s'\ 
     493                                  % str(e)) 
    494494 
    495495        # Check class inherits from CredWallet.CredRepos abstract base class 
    496496        if not issubclass(credReposClass, CredRepos): 
    497             raise SessionMgrError, \ 
    498                 "Credential Repository class %s must be inherited from %s" % \ 
    499                 (credReposClass, CredRepos) 
     497            raise SessionMgrError("Credential Repository class %s must be " 
     498                                  "inherited from %s" % \ 
     499                                  (credReposClass, CredRepos)) 
    500500 
    501501        # Instantiate custom class 
     
    506506             
    507507        except Exception, e: 
    508             raise SessionMgrError, \ 
    509             "Error instantiating Credential Repository interface: " + str(e) 
     508            raise SessionMgrError( 
     509            "Error instantiating Credential Repository interface: " + str(e)) 
    510510      
    511511        log.info(\ 
     
    523523    def __delitem__(self, key): 
    524524        "Session Manager keys cannot be removed"         
    525         raise KeyError, 'Keys cannot be deleted from '+self.__class__.__name__ 
     525        raise KeyError('Keys cannot be deleted from '+self.__class__.__name__) 
    526526 
    527527 
     
    531531        """ 
    532532        if key not in self.__prop: 
    533             raise KeyError, "Invalid key '%s'" % key 
     533            raise KeyError("Invalid key '%s'" % key) 
    534534         
    535535        return self.__prop[key] 
     
    545545 
    546546    def clear(self): 
    547         raise KeyError, "Data cannot be cleared from "+self.__class__.__name__ 
     547        raise KeyError("Data cannot be cleared from "+self.__class__.__name__) 
    548548    
    549549    def keys(self): 
     
    583583                log.debug('Set properties file path %s from "NDGSEC_DIR"'%val) 
    584584            else: 
    585                 raise AttributeError, 'Unable to set default Session ' + \ 
    586                     'Manager properties file path: neither ' + \ 
    587                     '"NDGSEC_SM_PROPFILEPATH" or "NDGSEC_DIR" environment ' + \ 
    588                     'variables are set' 
     585                raise AttributeError('Unable to set default Session ' 
     586                                     'Manager properties file path: neither '  
     587                                     '"NDGSEC_SM_PROPFILEPATH" or "NDGSEC_DIR"' 
     588                                     ' environment variables are set') 
    589589        else: 
    590590             log.debug('Set properties file path %s from user input' % val)        
    591591 
    592592        if not isinstance(val, basestring): 
    593             raise AttributeError, "Input Properties file path " + \ 
    594                                   "must be a valid string." 
     593            raise AttributeError("Input Properties file path must be a valid " 
     594                                 "string.") 
    595595       
    596596        self.__propFilePath = val 
     
    618618 
    619619            except IOError, e: 
    620                 raise SessionMgrError, \ 
     620                raise SessionMgrError( 
    621621                                "Error parsing properties file \"%s\": %s" % \ 
    622                                 (e.filename, e.strerror)                
     622                                (e.filename, e.strerror))               
    623623            except Exception, e: 
    624                 raise SessionMgrError, \ 
     624                raise SessionMgrError( 
    625625                    "Error parsing properties file: \"%s\": %s" % \ 
    626                     (self.__propFilePath, e) 
     626                    (self.__propFilePath, e)) 
    627627 
    628628        if propElem is None: 
    629             raise SessionMgrError, \ 
    630                             "Parsing properties: root element is not defined" 
     629            raise SessionMgrError( 
     630                            "Parsing properties: root element is not defined") 
    631631  
    632632        missingElem = [] 
     
    680680                 
    681681        except Exception, e: 
    682             raise SessionMgrError, \ 
    683                 'Error parsing tag "%s" in properties file: %s' % (elem.tag,e) 
     682            raise SessionMgrError( 
     683                'Error parsing tag "%s" in properties file: %s' % (elem.tag,e)) 
    684684 
    685685        missingElem.extend(getMissingElem(self.__prop, self.__validElem)) 
     
    693693 
    694694        if errMsg: 
    695             raise SessionMgrError, errMsg +  " for properties file" 
     695            raise SessionMgrError(errMsg +  " for properties file") 
    696696 
    697697        log.info('Loaded properties from "%s"' % self.__propFilePath) 
     
    707707        for key in prop.keys(): 
    708708            if key not in self.__validElem: 
    709                 raise SessionMgrError, "Property name \"%s\" is invalid" % key 
     709                raise SessionMgrError("Property name \"%s\" is invalid" % key) 
    710710 
    711711 
     
    726726                        self.__prop[key] = value               
    727727            else: 
    728                 raise SessionMgrError, \ 
    729                     "Key \"%s\" is not a valid Session Manager property" % key 
     728                raise SessionMgrError(\ 
     729                    "Key \"%s\" is not a valid Session Manager property" % key) 
    730730         
    731731         
     
    773773     
    774774     
    775     #_________________________________________________________________________         
    776775    def getSessionStatus(self, sessID=None, userDN=None): 
    777776        """Check the status of a given session identified by sessID or  
     
    789788        # Look for a session corresponding to this ID 
    790789        if sessID and userDN: 
    791             raise SessionMgrError, \ 
    792                             'Only "SessID" or "userDN" keywords may be set' 
     790            raise SessionMgrError('Only "SessID" or "userDN" keywords may be ' 
     791                                  'set') 
    793792        elif sessID: 
    794793            if sessID in self.__sessDict:                
     
    804803            try: 
    805804                # Enables re-ordering of DN fields for following dict search 
    806                 userDN = str(X509DN(userDN)) 
     805                userDN = str(X500DN(userDN)) 
    807806                 
    808807            except Exception, e: 
    809                 raise SessionMgrError, \ 
    810                 "Parsing input user certificate DN for getSessionStatus: %s"%e 
    811  
     808                log.error("Parsing input user certificate DN for " 
     809                          "getSessionStatus: %s" % e) 
     810                raise 
     811             
    812812            if userDN in self.__dnDict: 
    813813                log.info("Session found with DN = %s" % userDN) 
     
    890890                 
    891891            except Exception, e: 
    892                 raise SessionMgrError, "Delegating from MyProxy: %s" % e 
     892                raise SessionMgrError("Delegating from MyProxy: %s" % e) 
    893893 
    894894            if createServerSess: 
     
    923923             
    924924        except Exception, e: 
    925             raise SessionMgrError, \ 
     925            raise SessionMgrError(\ 
    926926                "Parsing input certificate DN for session create: %s" % \ 
    927                                                                     str(e) 
     927                                                                    str(e)) 
    928928 
    929929        if userDN in self.__dnDict: 
     
    951951                             *creds)       
    952952            except Exception, e: 
    953                 raise SessionMgrError, "Creating User Session: %s" % e 
     953                raise SessionMgrError("Creating User Session: %s" % e) 
    954954 
    955955            # Also allow access by user DN 
     
    961961        # Check for unique session ID 
    962962        if newSessID in self.__sessDict: 
    963             raise SessionMgrError, \ 
    964                 "New Session ID is already in use:\n\n %s" % newSessID 
     963            raise SessionMgrError( 
     964                "New Session ID is already in use:\n\n %s" % newSessID) 
    965965 
    966966        # Add new session to list                  
     
    10071007                 
    10081008            except Exception, e: 
    1009                 raise SessionMgrError, \ 
    1010                 "Parsing input user certificate DN for session connect: %s" %e 
     1009                raise SessionMgrError( 
     1010                "Parsing input user certificate DN for session connect: %s" %e) 
    10111011 
    10121012            try: 
     
    10261026                 
    10271027            except Exception, e: 
    1028                 raise SessionMgrError, \ 
    1029                 "Parsing input user certificate DN for session connect: %s" %e 
     1028                raise SessionMgrError(\ 
     1029                "Parsing input user certificate DN for session connect: %s" %e) 
    10301030             
    10311031            try: 
     
    10501050            # ! Delete user session since it's user certificate is invalid 
    10511051            self.deleteUserSession(userSess=userSess) 
    1052             raise UserSessionX509CertNotBeforeTimeError, \ 
    1053                                     "User session is invalid: %s" % e           
     1052            raise UserSessionX509CertNotBeforeTimeError( 
     1053                                    "User session is invalid: %s" % e)         
    10541054     
    10551055        except X509CertExpired, e: 
     
    10901090                 
    10911091            except KeyError: 
    1092                 raise SessionMgrError, \ 
    1093                     "Deleting user session - no matching session ID exists" 
     1092                raise SessionMgrError( 
     1093                    "Deleting user session - no matching session ID exists") 
    10941094 
    10951095            # Get associated user Distinguished Name 
     
    11011101                 
    11021102            except Exception, e: 
    1103                 raise SessionMgrError, \ 
     1103                raise SessionMgrError( 
    11041104                "Parsing input proxy certificate DN for session connect: %s"%\ 
    1105                                                                         str(e) 
     1105                                                                        str(e)) 
    11061106            try: 
    11071107                userSess = self.__dnDict[userDN] 
     
    11091109            except KeyError: 
    11101110                # User session not found with given proxy cert 
    1111                 raise SessionMgrError, \ 
    1112                     "No user session found matching input proxy certificate" 
     1111                raise SessionMgrError( 
     1112                    "No user session found matching input proxy certificate") 
    11131113         
    11141114        if userSess: 
     
    11161116        else: 
    11171117            # User session not found with given ID 
    1118             raise SessionMgrError, \ 
    1119                     '"sessID", "userCert" or "userSess" keywords must be set' 
     1118            raise SessionMgrError( 
     1119                    '"sessID", "userCert" or "userSess" keywords must be set') 
    11201120  
    11211121        # Delete associated sessions 
     
    11311131         
    11321132        except Exception, e: 
    1133             raise SessionMgrError, "Deleting user session: %s" % e         
     1133            raise SessionMgrError("Deleting user session: %s" % e)   
    11341134 
    11351135        log.info("Deleted user session: user DN = %s, sessID = %s" % \ 
     
    12361236                                 signingPriKeyPwd=self.__prop['keyPwd'])            
    12371237        except Exception, e: 
    1238             raise SessionMgrError, \ 
     1238            raise SessionMgrError( 
    12391239                "Re-directing attribute certificate request to \"%s\": %s" % \ 
    1240                 (userSessMgrURI, str(e)) 
     1240                (userSessMgrURI, str(e))) 
    12411241 
    12421242             
     
    12491249         
    12501250        except Exception, e: 
    1251             raise SessionMgrError, \ 
     1251            raise SessionMgrError( 
    12521252        "Forwarding Authorisation request for Session Manager \"%s\": %s" %\ 
    1253                 (userSessMgrURI, e) 
     1253                (userSessMgrURI, e)) 
    12541254 
    12551255 
Note: See TracChangeset for help on using the changeset viewer.