Froxlor icon indicating copy to clipboard operation
Froxlor copied to clipboard

[WIP] Extract mail log parsers into dedicated handler classes

Open discordier opened this issue 5 years ago • 1 comments

Description

This cleans up the traffic parser classes.

This PR is currently just a preview (hence marked as WIP and none of the boxes below checked despite having added unit tests).

The TrafficCron is a big bunch of code and the mail log parser etc. are as well. As discussed on IRC, I started restructuring them into smaller classes that are more easy to track.

Type of change

  • [X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Includes unit tests for:

  • [X] postfix log parsing
  • [X] dovecot log parsing

Checklist:

  • [ ] I have performed a self-review of my own code
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [ ] My changes generate no new warnings
  • [ ] I have added tests that prove my fix is effective or that my feature works
  • [ ] New and existing unit tests pass locally with my changes

discordier avatar Sep 16 '20 19:09 discordier

is this still being worked on?

d00p avatar Jul 21 '21 17:07 d00p

no feedback

d00p avatar Nov 05 '22 08:11 d00p