Changeset 416


Ignore:
Timestamp:
07/21/10 08:23:20 (11 years ago)
Author:
bas
Message:

Some more logging imporvements

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/email2trac.py.in

    r415 r416  
    365365                #
    366366                if self.DROP_SPAM and spam:
    367                         if self.parameters.debug > 2 :
    368                                 set.logger.debug('Message is a SPAM. Automatic ticket insertion refused (SPAM level > %d)' % self.SPAM_LEVEL)
    369 
     367
     368                        set.logger.info('Message is a SPAM. Automatic ticket insertion refused (SPAM level > %d)' % self.SPAM_LEVEL)
    370369                        return 'drop'   
    371370
     
    373372
    374373                        return 'Spam'   
    375 
    376374                else:
    377375
     
    394392
    395393                except KeyError, detail:
    396                         if self.parameters.debug > 2 :
    397                                 self.logger.debug('%s not defined, all messages are allowed.' %(keyword))
     394                        self.logger.debug('%s not defined, all messages are allowed.' %(keyword))
    398395
    399396                        return default
     
    505502
    506503                                if addr == self.trac_smtp_from:
    507                                         if self.parameters.debug:
    508                                                 self.logger.debug("Skipping %s mail address for CC-field" %(addr))
     504                                        self.logger.debug("Skipping %s mail address for CC-field" %(addr))
    509505                                        continue
    510506
     
    515511
    516512                        if email_cc:
    517                                 if self.parameters.debug:
    518                                                 self.logger.debug('set_reply_fields: %s' %email_cc)
     513                                self.logger.debug('set_reply_fields: %s' %email_cc)
    519514
    520515                                ticket['cc'] = self.email_to_unicode(email_cc)
     
    709704
    710705                if tkt['reporter'].lower() in [self.author, self.email_addr]:
    711                         if self.parameters.debug:
    712                                 self.logger.debug('ALLOW, %s is the ticket reporter' %(self.email_addr))
     706                        self.logger.debug('ALLOW, %s is the ticket reporter' %(self.email_addr))
    713707
    714708                        return True
     
    716710                perm = PermissionSystem(self.env)
    717711                if perm.check_permission('TICKET_MODIFY', self.author):
    718                         if self.parameters.debug:
    719                                 self.logger.debug('ALLOW, %s has trac permission to update the ticket' %(self.author))
     712                        self.logger.debug('ALLOW, %s has trac permission to update the ticket' %(self.author))
    720713
    721714                        return True
     
    730723                        for cc in cc_list:
    731724                                if self.email_addr.lower() in cc.strip():
    732 
    733                                         if self.parameters.debug:
    734                                                 self.logger.debug('ALLOW, %s is in the CC' %(self.email_addr))
     725                                        self.logger.debug('ALLOW, %s is in the CC' %(self.email_addr))
    735726
    736727                                        return True
     
    811802
    812803                                if self.trac_smtp_from in cc_list:
    813                                         if self.parameters.debug:
    814                                                 self.logger.debug('TD: MAIL LOOP: %s is not allowed as CC address' %(self.trac_smtp_from))
     804                                        self.logger.debug('TD: MAIL LOOP: %s is not allowed as CC address' %(self.trac_smtp_from))
    815805
    816806                                        cc_list.remove(self.trac_smtp_from)
     
    930920                                fields = controller.get_ticket_changes(req, tkt, self.WORKFLOW)
    931921
    932                                 if self.parameters.debug:
    933                                         self.logger.debug('Workflow ticket update fields: ')
     922                                self.logger.debug('Workflow ticket update fields: ')
    934923
    935924                                for key in fields.keys():
    936                                         if self.parameters.debug:
    937                                                 self.logger.debug('\t %s : %s' %(key, fields[key]))
    938 
     925                                        self.logger.debug('\t %s : %s' %(key, fields[key]))
    939926                                        tkt[key] = fields[key]
    940927
     
    1006993       
    1007994                        if self.parameters.debug:
    1008                                 s = 'TD: trac.ini name %s = %s' %(name, value)
     995                                s = 'trac[%s] = %s' %(name, value)
    1009996                                self.print_unicode(s)
    1010997
     
    10151002                                value = self.parameters['%s_%s' %(prefix, name)]
    10161003                                if self.parameters.debug > 10:
    1017                                         s = 'TD: email2trac.conf %s = %s ' %(name, value)
     1004                                        s = 'email2trac[%s] = %s ' %(name, value)
    10181005                                        self.print_unicode(s)
    10191006
     
    10221009               
    10231010                        if self.parameters.debug:
    1024                                 s = 'TD: user_dict[%s] = %s' %(name, value)
     1011                                s = 'used %s = %s' %(name, value)
    10251012                                self.print_unicode(s)
    10261013
     
    10551042                                orig_subject = result.group(3)
    10561043
    1057                                 if self.parameters.debug:
    1058                                         self.logger.debug('subject search string: %s' %(orig_subject))
     1044                                self.logger.debug('subject search string: %s' %(orig_subject))
    10591045
    10601046                                cursor = self.db.cursor()
     
    10681054
    10691055                                for summary in summaries:
    1070                                         if self.parameters.debug:
    1071                                                 self.logger.debug('Looking for summary matching: "%s"' % summary)
     1056                                        self.logger.debug('Looking for summary matching: "%s"' % summary)
    10721057
    10731058                                        sql = """SELECT id FROM ticket
     
    10791064                                                (matched_id,) = row
    10801065
    1081                                                 if self.parameters.debug:
    1082                                                         self.logger.debug('Found matching ticket id: %d' % matched_id)
     1066                                                self.logger.debug('Found matching ticket id: %d' % matched_id)
    10831067
    10841068                                                break
     
    13351319                from trac.util.datefmt import to_timestamp, utc
    13361320
    1337                 if self.parameters.debug:
    1338                         self.logger.debug('Creating a new topic in forum:', self.id)
     1321                self.logger.debug('Creating a new topic in forum:', self.id)
    13391322
    13401323                # Get dissussion API component.
     
    13451328                forum = api.get_forum(context, self.id)
    13461329
    1347                 if not forum and self.parameters.debug:
    1348                         self.logger.debug("ERROR: Replied forum doesn't exist")
     1330                if not forum:
     1331                        self.logger.error("ERROR: Replied forum doesn't exist")
    13491332
    13501333                # Prepare topic.
     
    13661349                from trac.util.datefmt import to_timestamp, utc
    13671350
    1368                 if self.parameters.debug:
    1369                         self.logger.debug('Replying to discussion topic', self.id)
     1351                self.logger.debug('Replying to discussion topic', self.id)
    13701352
    13711353                # Get dissussion API component.
     
    13761358                topic = api.get_topic(context, self.id)
    13771359
    1378                 if not topic and self.parameters.debug:
    1379                         self.logger.debug("ERROR: Replied topic doesn't exist")
     1360                if not topic:
     1361                        self.logger.error("ERROR: Replied topic doesn't exist")
    13801362
    13811363                # Prepare message.
     
    13971379                from trac.util.datefmt import to_timestamp, utc
    13981380
    1399                 if self.parameters.debug:
    1400                         self.loggger.debug('TD: Replying to discussion message', self.id)
     1381                self.loggger.debug('TD: Replying to discussion message', self.id)
    14011382
    14021383                # Get dissussion API component.
     
    14071388                message = api.get_message(context, self.id)
    14081389
    1409                 if not message and self.parameters.debug:
    1410                         self.logger.debug("ERROR: Replied message doesn't exist")
     1390                if not message:
     1391                        self.logger.error("ERROR: Replied message doesn't exist")
    14111392
    14121393                # Prepare message.
     
    15291510               
    15301511                if not m:
    1531                         if self.parameters.debug:
    1532                                 self.logger.debug('This is not a valid email message format')
    1533 
     1512                        self.logger.debug('This is not a valid email message format')
    15341513                        return
    15351514                       
     
    15471526
    15481527                if not self.email_header_acl('white_list', self.email_addr, True):
    1549                         if self.parameters.debug:
    1550                                 self.logger.debug('Message rejected : %s not in white list' %(self.email_addr))
    1551 
     1528                        self.logger.info('Message rejected : %s not in white list' %(self.email_addr))
    15521529                        return False
    15531530
    15541531                if self.email_header_acl('black_list', self.email_addr, False):
    1555                         if self.parameters.debug:
    1556                                 self.logger.debug('Message rejected : %s in black list' %(self.email_addr))
    1557 
     1532                        self.logger.info('Message rejected : %s in black list' %(self.email_addr))
    15581533                        return False
    15591534
    15601535                if not self.email_header_acl('recipient_list', self.to_email_addr, True):
    1561                         if self.parameters.debug:
    1562                                 self.logger.debug('Message rejected : %s not in recipient list' %(self.to_email_addr))
    1563 
     1536                        self.logger.info('Message rejected : %s not in recipient list' %(self.to_email_addr))
    15641537                        return False
    15651538
     
    15791552                        subject  = self.email_to_unicode(m['Subject'])
    15801553
    1581                 if self.parameters.debug:
    1582                         self.logger.debug('subject: %s' %subject)
     1554                self.logger.debug('subject: %s' %subject)
    15831555
    15841556                #
     
    17541726                                self.properties[keyword] = value.strip()
    17551727
    1756                                 if self.parameters.debug:
    1757                                         self.logger.debug('inline properties: %s : %s' %(keyword,value))
     1728                                self.logger.debug('inline properties: %s : %s' %(keyword,value))
     1729
    17581730                        else:
    17591731                                body.append(line)
     
    18071779
    18081780                for part in msg.walk():
    1809                         if self.parameters.debug:
    1810                                 self.logger.debug('Message part: Main-Type: %s' % part.get_content_maintype())
    1811                                 self.logger.debug('Message part: Content-Type: %s' % part.get_content_type())
     1781                        self.logger.debug('Message part: Main-Type: %s' % part.get_content_maintype())
     1782                        self.logger.debug('Message part: Content-Type: %s' % part.get_content_type())
    18121783
    18131784                        ## Check content type
    18141785                        #
    18151786                        if part.get_content_type() in self.STRIP_CONTENT_TYPES:
    1816 
    1817                                 if self.parameters.debug:
    1818                                         self.logger.debug("A %s attachment named '%s' was skipped" %(part.get_content_type(), part.get_filename()))
    1819 
     1787                                self.logger.debug("A %s attachment named '%s' was skipped" %(part.get_content_type(), part.get_filename()))
    18201788                                continue
    18211789
     
    18291797                        #
    18301798                        if part.get_content_maintype() == 'multipart':
    1831                                 if self.parameters.debug:
    1832                                         self.logger.debug("Skipping multipart container")
     1799                                self.logger.debug("Skipping multipart container")
     1800
    18331801                                continue
    18341802                       
     
    18411809                        if ALTERNATIVE_MULTIPART and self.DROP_ALTERNATIVE_HTML_VERSION:
    18421810                                if part.get_content_type() == 'text/html':
    1843                                         if self.parameters.debug:
    1844                                                 self.logger.debug('Skipping alternative HTML message')
    1845 
    1846                                         ALTERNATIVE_MULTIPART = False
     1811                                        self.logger.debug('Skipping alternative HTML message')
     1812                                        ALTERNATIVE_MULTIPART = False
    18471813                                        continue
    18481814
     
    18501816                        #
    18511817                        if part.get_content_type() == 'text/plain' and inline:
    1852                                 if self.parameters.debug:
    1853                                         self.logger.debug('               Inline body part')
     1818                                self.logger.debug('               Inline body part')
    18541819
    18551820                                # Try to decode, if fails then do not decode
Note: See TracChangeset for help on using the changeset viewer.