Changeset 518


Ignore:
Timestamp:
04/21/11 11:00:29 (13 years ago)
Author:
bas
Message:

Fixed comparison of email addresses and author names. Always do lower case comparison, closes #242

Location:
trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/ChangeLog

    r517 r518  
    112.X.X
     2  * Convert all email addresses and author names to lower case in compare
     3    functions.  closes #242.
     4    Author: Dennis McRitchie <dmcr add Princeton dot EDU >
     5    Applied by: Bas van der Vlies
     6
    27  * Added timestamp when log_file stderr or file is used. Move log level of
    38    reporting subject line to info instead of debug, closes #236
  • trunk/debian/changelog

    r517 r518  
     1email2trac (2.0.3-2) stable; urgency=low
     2
     3  * see Changelog
     4
     5 -- bas van der Vlies <basv@sara.nl>  Thu, 21 Apr 2011 10:54:37 +0200
     6
    17email2trac (2.0.3-1) stable; urgency=low
    28
    39  * See ChangeLog
    410
    5  -- bas van der Vlies <bas@test3.irc.sara.nl.irc.sara.nl>  Wed, 23 Mar 2011 10:06:56 +0100
     11 -- bas van der Vlies <basv@sara.nl>  Wed, 23 Mar 2011 10:06:56 +0100
    612
    713email2trac (2.0.2-1) stable; urgency=low
  • trunk/email2trac.py.in

    r517 r518  
    326326        ## Trac can not handle author's name that contains spaces
    327327        #
    328         if self.email_addr == self.trac_smtp_from:
     328        if self.email_addr.lower() == self.trac_smtp_from.lower():
    329329            if self.email_name:
    330330                self.author = self.email_name
     
    346346            if (
    347347                (e and (e.lower() == self.email_addr.lower())) or
    348                 (u + '@' + self.smtp_default_domain.lower() == self.email_addr.lower())
     348                (u.lower() + '@' + self.smtp_default_domain.lower() == self.email_addr.lower())
    349349            )
    350350            ]
     
    648648        self.logger.debug('function check_permission_participants')
    649649
    650         if tkt['reporter'].lower() in [self.author, self.email_addr]:
     650        if tkt['reporter'].lower() in [self.author.lower(), self.email_addr.lower()]:
    651651            self.logger.debug('ALLOW, %s is the ticket reporter' %(self.email_addr))
    652652
     
    663663            cc_list = tkt['cc'].split(',')
    664664            for cc in cc_list:
    665                 if self.email_addr.lower() in cc.strip():
     665                if self.email_addr.lower() in cc.lower.strip():
    666666                    self.logger.debug('ALLOW, %s is in the CC' %(self.email_addr))
    667667
Note: See TracChangeset for help on using the changeset viewer.