From d7f2e5bcc474af1733d7e3ad27eb2e926ebf3f51 Mon Sep 17 00:00:00 2001
From: Tomas Cohen Arazi <tomascohen@theke.io>
Date: Fri, 5 Nov 2021 12:50:35 -0300
Subject: [PATCH] Bug 29427: Make SMTP transport set debug mode correctly

This patch makes the _transport_ method in Koha::SMTP::Server pass the
debug flag correctly to the Email::Sender::Transport::SMTP->new
constructor.

To test:
1. Apply the regression tests patch
2. Run:
   $ kshell
  k$ prove t/db_dependent/Koha/SMTP/Server.t
=> FAIL: Oh noes! debug is not passed to the constructor!
3. Apply this patch
4. Repeat 2
=> SUCCESS: Hey! It is being set!
5. Sign off :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit b0767f94d07950e7420deb901b208d131427c535)

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
---
 Koha/SMTP/Server.pm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Koha/SMTP/Server.pm b/Koha/SMTP/Server.pm
index d5fec72611..7a2a40cb18 100644
--- a/Koha/SMTP/Server.pm
+++ b/Koha/SMTP/Server.pm
@@ -64,6 +64,7 @@ sub transport {
     $params->{sasl_password} = $self->password
         if $self->password;
 
+    $params->{debug} = $self->debug;
 
     my $transport = Email::Sender::Transport::SMTP->new( $params );
 
-- 
2.39.5