Modify

Ticket #1453 (closed defect: fixed)

Opened 4 years ago

Last modified 3 years ago

Enhance mail logs

Reported by: javier.amor.garcia@… Owned by: javier.amor.garcia@…
Milestone: 1.4 Component: mail
Severity: normal Keywords:
Cc: iclerencia@…

Description (last modified by javier.amor.garcia@…) (diff)

This is neccessay for reportin in #1451

  • discriminate between mail sent and received
  • consolidate logs

Attachments

Change History

comment:1 Changed 4 years ago by javier.amor.garcia@…

  • Description modified (diff)

comment:2 Changed 4 years ago by javier.amor.garcia@…

(In [14774]) NN dapted to changes inpostfix years dates refs #1453

comment:3 Changed 4 years ago by javier.amor.garcia@…

(In [14778]) added log ocnsolidation and report method to mail refs #1451

comment:4 Changed 4 years ago by javier.amor.garcia@…

  • Cc iclerencia@… added

Mail now has consolidation of logs. however I think that MailLogObserver? is not working properly this is the reason I havent implemented the discrimination between sent/received at 'full log' level, I hadn't time to fix MailLogObserver?.

One case where I am sure it does not work properly is when we send the mail to other SMTP server and is rejected for whatever reason (i have witnessed it with greylisting) i nthis case ther isn't any log for this event.

The bottom line is that we have to revise and fix LogObnserver? whenever possible.

comment:5 Changed 4 years ago by javier.amor.garcia@…

(In [14814]) backported fix in logs from trunk. It seems that solves the bugs reproted in this ticket refs #1453

comment:6 Changed 3 years ago by javier.amor.garcia@…

(In [15582]) BP: now if more than one LogHelper? uses the same fiel the lines are only aprsed one time closes #1436 refs #1453

comment:7 Changed 3 years ago by anonymous

Changeset [15585] by javier.amor.garcia

NN: added more cases

comment:8 Changed 3 years ago by anonymous

  • Status changed from new to closed
  • Resolution set to fixed

We have fixed know errors, tested cases and the whole thing seems in place. We close it until new error are found.

comment:9 Changed 3 years ago by javier.amor.garcia@…

(In [15609]) NN: added migration to change not null status in one of the mail table log columns refs #1453

comment:10 Changed 3 years ago by javier.amor.garcia@…

(In [15580]) fixed bad test refs #1623

comment:11 Changed 3 years ago by javier.amor.garcia@…

  • Milestone set to 1.4

comment:12 Changed 3 years ago by javier.amor.garcia@…

(In [15712]) NN: fixed to field for alis mai lrecipients refs #1453

comment:13 Changed 3 years ago by javier.amor.garcia@…

(In [15754]) NN: adapted test to change in years refs #1453

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.