Changeset 30385

Show
Ignore:
Timestamp:
12/07/05 16:44:16 (4 years ago)
Author:
tziade
Message:

protecting from header recipient (#1121)

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • CPSMailAccess/trunk/mailmessageeditview.py

    r28287 r30385  
    220220    def _identyToMsgHeader(self): 
    221221        """ takes a directory entry to fit it in From header """ 
     222        def cleanEntry(entry): 
     223            for element in (' ', '<', '>', ' '): 
     224                entry = entry.strip(element) 
     225            return entry 
     226 
    222227        identities = self.getIdentitites() 
    223228        if len(identities) > 0: 
     
    226231                email = '?' 
    227232            else: 
    228                 email = identity['email'].strip() 
     233                email = cleanEntry(identity['email']) 
     234 
    229235            if identity['fullname'] != '': 
    230                 return '%s <%s>' %(identity['fullname'], email) 
     236                fullname = cleanEntry(identity['fullname']) 
     237                return '%s <%s>' %(fullname, email) 
    231238            else: 
    232239                return email 
  • CPSMailAccess/trunk/tests/test_mailmessageeditview.py

    r28769 r30385  
    244244        self.assertEquals(url, 'nowhere/INBOX/INBOX') 
    245245 
     246    def test_identyToMsgHeader(self): 
     247        def getIdentitites(): 
     248            return ({'email': '< tz@nuxeo.com>', 'fullname': '<tarek> '},) 
     249        mailbox = self._getMailBox() 
     250        view = MailMessageEdit(mailbox, None) 
     251        view.getIdentitites = getIdentitites 
     252 
     253        msg_from = view._identyToMsgHeader() 
     254        self.assertEquals(msg_from, 'tarek <tz@nuxeo.com>') 
     255 
    246256def test_suite(): 
    247257    return unittest.TestSuite((