[pmg-devel] [PATCH pmg-api/pmg-log-tracker v2] add dedicated before queue status
Stoiko Ivanov
s.ivanov at proxmox.com
Thu Apr 30 17:01:48 CEST 2020
chagnes v1->v2:
* added short comment based on Mira's review (thanks!)
* added R-b, T-b tags from the review mails.
while trying to add a new test-case (when pmg-smtp-filter yields different
results for differnt recipients in the before-queue case) I noticed a small
glitch - however this has been present all the way back to the first version
with before-queue filtering support - and will be fixed independently - see
#2713
cover-letter from v1:
This patchset aims to provide the same output of the API for the status
of before-queue and after-queue filtered mails.
Since the current output of pmg-log-tracker does not provide enough information
in the before-queue case (apart from successfully delivered downstream mails),
I added 3 new status-values for before-queue filtered mails, which are then
translated in the pmg-api to the same values as in the after-queue case.
This is my first patch to Rust code and is mostly due to copy is my hobby, so
an experienced look would be very much appreciated!
pmg-api:
Stoiko Ivanov (1):
MailTracker: handle before queue status
src/PMG/API2/MailTracker.pm | 18 ++++++++++++++----
1 file changed, 14 insertions(+), 4 deletions(-)
pmg-log-tracker:
Stoiko Ivanov (2):
Introduce before queue dstatus
add test for before queue dstatus
src/main.rs | 21 +++++++++++-
tests/test_input_mixed | 32 +++++++++++++++++++
tests/test_output_before_queue | 8 ++---
tests/test_output_before_queue_host | 8 ++---
.../test_output_before_queue_mixed_downstream | 21 ++++++++++++
tests/test_output_before_queue_qid | 8 ++---
tests/tests_before_queue.rs | 21 ++++++++++++
7 files changed, 106 insertions(+), 13 deletions(-)
create mode 100644 tests/test_output_before_queue_mixed_downstream
--
2.20.1
More information about the pmg-devel
mailing list