[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Magnus Enger changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=36115 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Victor Grousset/tuxayo changed: What|Removed |Added Status|Pushed to oldstable |RESOLVED CC||vic...@tuxayo.net Resolution|--- |FIXED --- Comment #62 from Victor Grousset/tuxayo --- Missing dependencies for 20.05.x, it shouldn't be affected, no backport. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Fridolin Somers changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m Version(s)|21.11.00,21.05.05 |21.11.00,21.05.05,20.11.12 released in|| Status|Pushed to stable|Pushed to oldstable --- Comment #61 from Fridolin Somers --- Pushed to 20.11.x for 20.11.12 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Kyle M Hall changed: What|Removed |Added CC||k...@bywatersolutions.com Version(s)|21.11.00|21.11.00,21.05.05 released in|| Status|Pushed to master|Pushed to stable --- Comment #60 from Kyle M Hall --- Pushed to 21.05.x for 21.05.05 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #59 from Jonathan Druart --- Pushed to master for 21.11, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||21.11.00 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Marcel de Rooy changed: What|Removed |Added Depends on||22343 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22343 [Bug 22343] Add configuration options for SMTP servers -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729
Marcel de Rooy changed:
What|Removed |Added
Status|BLOCKED |Passed QA
--- Comment #58 from Marcel de Rooy ---
(In reply to Marcel de Rooy from comment #54)
> We cover ReturnPathDefault but do not fallback to Admin like here:
>
> +$args->{from} = $self->email->header("Sender") ||
> C4::Context->preference('KohaAdminEmailAddress');
>
> I am keeping it in blocked over the weekend..
Dont think we should do that here.
If you keep the pref ReturnpathDefault empty, return path falls back correctly
to sender. And we already use AdminEmail as a fallback.
The fix is fine as-is.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #57 from Marcel de Rooy --- (In reply to Martin Renvoize from comment #55) > (In reply to Marcel de Rooy from comment #53) > > The fix works fine on itself, but it feels like a fix on the wrong place. > > And a fix that more or less assumes that we are not calling send_or_die with > > the from parameter; this seems to the case in the current codebase where we > > only pass transport. > > Not sure what you mean here.. we account for 'from' being passed... the key > challenge is understanding that the 'from' you might pass send_or_die is the > envelope address and not the email from. > > > We should be fixing create too (origin of this problem). I will postpone > > these things to 29365. The test passes, could be improved too. > > > > This serves as a minimal fix, allowing backports. > > I'm not sure what you mean about fixing create? Sender is a bad name for > this parameter and it was used due to a weird internal workaround provided > by Mail::Sendmail.. I don't know that you could easily move the fix into > create.. short of create producing the wrapper and returning a structure > that's all nested one level deeper. Lets see on the new report. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #56 from Martin Renvoize --- Created attachment 127094 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127094=edit Bug 28729: Additions to POD -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #55 from Martin Renvoize --- (In reply to Marcel de Rooy from comment #53) > The fix works fine on itself, but it feels like a fix on the wrong place. > And a fix that more or less assumes that we are not calling send_or_die with > the from parameter; this seems to the case in the current codebase where we > only pass transport. Not sure what you mean here.. we account for 'from' being passed... the key challenge is understanding that the 'from' you might pass send_or_die is the envelope address and not the email from. > We should be fixing create too (origin of this problem). I will postpone > these things to 29365. The test passes, could be improved too. > > This serves as a minimal fix, allowing backports. I'm not sure what you mean about fixing create? Sender is a bad name for this parameter and it was used due to a weird internal workaround provided by Mail::Sendmail.. I don't know that you could easily move the fix into create.. short of create producing the wrapper and returning a structure that's all nested one level deeper. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729
--- Comment #54 from Marcel de Rooy ---
We cover ReturnPathDefault but do not fallback to Admin like here:
+$args->{from} = $self->email->header("Sender") ||
C4::Context->preference('KohaAdminEmailAddress');
I am keeping it in blocked over the weekend..
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Marcel de Rooy changed: What|Removed |Added Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #53 from Marcel de Rooy --- The fix works fine on itself, but it feels like a fix on the wrong place. And a fix that more or less assumes that we are not calling send_or_die with the from parameter; this seems to the case in the current codebase where we only pass transport. We should be fixing create too (origin of this problem). I will postpone these things to 29365. The test passes, could be improved too. This serves as a minimal fix, allowing backports. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Marcel de Rooy changed: What|Removed |Added Attachment #127016|0 |1 is obsolete|| --- Comment #52 from Marcel de Rooy --- Created attachment 127086 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127086=edit Bug 28729: Unit tests Signed-off-by: David Nind Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729
Marcel de Rooy changed:
What|Removed |Added
Attachment #127017|0 |1
is obsolete||
--- Comment #51 from Marcel de Rooy ---
Created attachment 127085
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127085=edit
Bug 28729: Make Koha::Email->send_or_die handle returnpath
Koha used to rely on Mail::Sendmail for sending emails. As an SMTP
client, the library would extract the from address from the Sender
header to pass along in the MAIL FROM: field of the SMTP protocol [1].
This was overlooked when we moved to Email::Stuffer/Email::Simple and
there's a different behavior as it expects the envelope to be passed and
falls back to extracting the 'From' header when said envelope is not
found [2].
This patchset re-introduces the behavior from Mail::Sendmail by
overriding the send_or_die method locally (in Koha::Email) and doing the
right thing.
Unless an explicit {from} parameter is passed, it extracts the MAIL FROM
envelope from the Sender header, as Mail::Sendmail did, and calls
$self->SUPER::send_or_die with the right parameters.
To test:
1. Apply the unit tests
2. Run:
$ kshell
k$ prove t/Koha/Email.t
=> FAIL: Sender is not handled correctly!
3. Apply this patch
4. Repeat 2
=> SUCCESS: Tests pass!
The from parameter is correct!
No Sender header sent!
5. Sign off :-D
[1] https://metacpan.org/dist/Mail-Sendmail/source/lib/Mail/Sendmail.pm#L284
[2]
https://metacpan.org/pod/Email::Sender::Manual::QuickStart#envelope-information
Signed-off-by: David Nind
Signed-off-by: Marcel de Rooy
AMENDED (SHORTENED)
-my @headers = $self->email->header_str_pairs;
-foreach my $pair ( pairs @headers ) {
-my ( $header, $value ) = @$pair;
-$args->{from} = $value if $header eq 'Sender';
-}
-
-# Remove the Sender header
-$self->email->header_str_set('Sender');
+$args->{from} = $self->email->header_str('Sender');
+$self->email->header_str_set('Sender'); # remove Sender header
Tested with same results (scrambled domains):
From: nore...@mydevserver.com
Cc: mar...@email.nl
To: t...@somewhere.nl
Reply-To: b...@mydevserver.com
Return-Path: postmas...@mydevserver.com
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Marcel de Rooy changed: What|Removed |Added Blocks||29365 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29365 [Bug 29365] Koha::Email issues -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #50 from Marcel de Rooy --- Resuming tomorrow, dont worry -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Marcel de Rooy changed: What|Removed |Added Status|Signed Off |BLOCKED --- Comment #49 from Marcel de Rooy --- QA: Looking here -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729
David Nind changed:
What|Removed |Added
Attachment #126527|0 |1
is obsolete||
--- Comment #48 from David Nind ---
Created attachment 127017
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127017=edit
Bug 28729: Make Koha::Email->send_or_die handle returnpath
Koha used to rely on Mail::Sendmail for sending emails. As an SMTP
client, the library would extract the from address from the Sender
header to pass along in the MAIL FROM: field of the SMTP protocol [1].
This was overlooked when we moved to Email::Stuffer/Email::Simple and
there's a different behavior as it expects the envelope to be passed and
falls back to extracting the 'From' header when said envelope is not
found [2].
This patchset re-introduces the behavior from Mail::Sendmail by
overriding the send_or_die method locally (in Koha::Email) and doing the
right thing.
Unless an explicit {from} parameter is passed, it extracts the MAIL FROM
envelope from the Sender header, as Mail::Sendmail did, and calls
$self->SUPER::send_or_die with the right parameters.
To test:
1. Apply the unit tests
2. Run:
$ kshell
k$ prove t/Koha/Email.t
=> FAIL: Sender is not handled correctly!
3. Apply this patch
4. Repeat 2
=> SUCCESS: Tests pass!
The from parameter is correct!
No Sender header sent!
5. Sign off :-D
[1] https://metacpan.org/dist/Mail-Sendmail/source/lib/Mail/Sendmail.pm#L284
[2]
https://metacpan.org/pod/Email::Sender::Manual::QuickStart#envelope-information
Signed-off-by: David Nind
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 David Nind changed: What|Removed |Added Attachment #126526|0 |1 is obsolete|| --- Comment #47 from David Nind --- Created attachment 127016 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127016=edit Bug 28729: Unit tests Signed-off-by: David Nind -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 David Nind changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Marcel de Rooy changed: What|Removed |Added QA Contact|tomasco...@gmail.com|m.de.r...@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Martin Renvoize changed: What|Removed |Added Severity|major |critical -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #46 from Katrin Fischer --- Actually, we did - comment#34. I missed it earlier too. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #45 from Martin Renvoize --- Wish you'd submitted that before I dug through it and came up with basically the same. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729
--- Comment #44 from Martin Renvoize ---
Created attachment 126527
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=126527=edit
Bug 28729: Make Koha::Email->send_or_die handle returnpath
Koha used to rely on Mail::Sendmail for sending emails. As an SMTP
client, the library would extract the from address from the Sender
header to pass along in the MAIL FROM: field of the SMTP protocol [1].
This was overlooked when we moved to Email::Stuffer/Email::Simple and
there's a different behavior as it expects the envelope to be passed and
falls back to extracting the 'From' header when said envelope is not
found [2].
This patchset re-introduces the behavior from Mail::Sendmail by
overriding the send_or_die method locally (in Koha::Email) and doing the
right thing.
Unless an explicit {from} parameter is passed, it extracts the MAIL FROM
envelope from the Sender header, as Mail::Sendmail did, and calls
$self->SUPER::send_or_die with the right parameters.
To test:
1. Apply the unit tests
2. Run:
$ kshell
k$ prove t/Koha/Email.t
=> FAIL: Sender is not handled correctly!
3. Apply this patch
4. Repeat 2
=> SUCCESS: Tests pass!
The from parameter is correct!
No Sender header sent!
5. Sign off :-D
[1] https://metacpan.org/dist/Mail-Sendmail/source/lib/Mail/Sendmail.pm#L284
[2]
https://metacpan.org/pod/Email::Sender::Manual::QuickStart#envelope-information
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #43 from Martin Renvoize --- Created attachment 126526 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=126526=edit Bug 28729: Unit tests -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #42 from Martin Renvoize --- I changed my mind.. patches inbound -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Martin Renvoize changed: What|Removed |Added Attachment #126436|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729
--- Comment #41 from Katrin Fischer ---
Hi Martin,
thanks a lot for looking into this! We'll try to test your fix, this is the
patch we have been running with meanwhile:
Set the envelope-from and therefore the return-path to sender.
---
Koha/Email.pm | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Koha/Email.pm b/Koha/Email.pm
index 9c077a6b84..068f8d9004 100644
--- a/Koha/Email.pm
+++ b/Koha/Email.pm
@@ -173,6 +173,9 @@ sub send_or_die {
$args->{to} = \@recipients;
}
+$args->{from} = $self->email->header("Sender") ||
C4::Context->preference('KohaAdminEmailAddress');
+Koha::Exceptions::BadParameter->throw("Invalid 'from' parameter:
".$args->{from})
+unless Email::Valid->address( -address => $args->{from}, -fqdn => 0 );
# from is mandatory
$self->SUPER::send_or_die($args);
}
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Martin Renvoize changed: What|Removed |Added Status|ASSIGNED|Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Martin Renvoize changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |tomasco...@gmail.com |y.org | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Martin Renvoize changed: What|Removed |Added Assignee|tomasco...@gmail.com|martin.renvoize@ptfs-europe ||.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #40 from Martin Renvoize --- Created attachment 126436 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=126436=edit Bug 28729: Reverse 'sender' and 'from' Having read through the docs for both Email::Stuffer (Email::Sender) and Mail::Sendmail, I believe the meaning of 'from' and 'Sender' has reversed between the two modules. Email::Sendmail interpreted 'from' as the from address of the mail and used Sender as an override for the envelope when it was passed. However, in Email::Stuffer the 'from' is set at the envelope level and thus sets the connection 'MAIL FROM' which is subsequently used as the return-path by the mail server. It then uses 'Sender' as the message from address if one is passed in as a header. This patch reverses our handling of those attribute in Koha::Email. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Martin Renvoize changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com --- Comment #39 from Martin Renvoize --- OK.. I've dug through this a bit.. man the Email::Stuffer and friends and Mail::Sendmail docs are hard to understand. In short, I think 'Sender' and 'From' are basically reversed between the two sets of modules.. I would say we need to set 'From' to 'returnpath' info (Thus setting the envelope MAIL FROM' used in the mail server connection and then if we have a more specific 'From' we want in the actual mail we set that in the 'Sender' slot. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729
--- Comment #38 from Katrin Fischer ---
> The default for Return-Path should be preference ReturnpathDefault.
> Is that still used then?
As we copy the Sender information that behaves correctly in my testing, I think
that is the case. You can find this line further above in Email.pm create()
$addresses->{sender} ||= C4::Context->preference('ReturnpathDefault')
if C4::Context->preference('ReturnpathDefault');
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #37 from Ulrich Kleiber --- The described change of the patch in my last comment is incorrect. Please ignore. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729
--- Comment #36 from Ulrich Kleiber ---
If no branchreturnpath exists, sender and thus envelope-from will be set to
ReturnpathDefault.
Maybe we should use
$args->{from} ||= $self->email->header("Sender");
instead of
$args->{from} = $self->email->header("Sender") ||
C4::Context->preference('KohaAdminEmailAddress');
in the upper patch.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729
--- Comment #35 from Marcel de Rooy ---
(In reply to Ulrich Kleiber from comment #34)
> With this patch only the envelope-from is changed. After that, emailing
> under 20.11 behaves much like emailing under 18.11. At least that's true for
> our installations.
>
> It's not pretty and I don't know if it covers everything or has any negative
> side effects. I hope this helps with further troubleshooting.
>
> --- Email.pm 2021-07-29 11:07:24.560924469 +0200
> +++ Email.pm.patched 2021-07-29 16:33:26.636468041 +0200
> @@ -173,6 +173,9 @@
> $args->{to} = \@recipients;
> }
>
> +$args->{from} = $self->email->header("Sender") ||
> C4::Context->preference('KohaAdminEmailAddress');
> +Koha::Exceptions::BadParameter->throw("Invalid 'from' parameter:
> ".$args->{from})
> +unless Email::Valid->address( -address => $args->{from}, -fqdn => 0
> ); # from is mandatory
> $self->SUPER::send_or_die($args);
> }
The default for Return-Path should be preference ReturnpathDefault.
Is that still used then?
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729
--- Comment #34 from Ulrich Kleiber ---
With this patch only the envelope-from is changed. After that, emailing under
20.11 behaves much like emailing under 18.11. At least that's true for our
installations.
It's not pretty and I don't know if it covers everything or has any negative
side effects. I hope this helps with further troubleshooting.
--- Email.pm2021-07-29 11:07:24.560924469 +0200
+++ Email.pm.patched2021-07-29 16:33:26.636468041 +0200
@@ -173,6 +173,9 @@
$args->{to} = \@recipients;
}
+$args->{from} = $self->email->header("Sender") ||
C4::Context->preference('KohaAdminEmailAddress');
+Koha::Exceptions::BadParameter->throw("Invalid 'from' parameter:
".$args->{from})
+unless Email::Valid->address( -address => $args->{from}, -fqdn => 0 );
# from is mandatory
$self->SUPER::send_or_die($args);
}
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #33 from Katrin Fischer --- > So this proves that if the MTA copies Sender to Return-path, Koha indirectly > sets Return-path :) > It is not enough to just look at headers leaving Koha; also look at what > happens between sender and receiving side. Thx for the test - could you maybe repeat on master/20.11 for checking on differences? If the MTA hasn't changed like in our case... Koha might send something slightly different? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #32 from Marcel de Rooy --- (In reply to Jonathan Druart from comment #18) > Tried on 18.11.x > > -- MESSAGE FOLLOWS -- > Sender: returnpathdefa...@default.org > Mime-Version: 1.0 > Charset: utf8 > Content-Transfer-Encoding: quoted-printable > Subject: =?UTF-8?B?Q2hlY2tvdXRz?= > Content-Type: text/plain; charset="UTF-8" > Date: Wed, 28 Jul 2021 12:59:39 + > To: e...@example.org > From: r...@example.org > X-Peer: 127.0.0.1 > > The following items have been checked out: > E Street shuffle=20 > Thank you for visiting Centerville.= > > END MESSAGE So this proves that if the MTA copies Sender to Return-path, Koha indirectly sets Return-path :) It is not enough to just look at headers leaving Koha; also look at what happens between sender and receiving side. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729
--- Comment #31 from Katrin Fischer ---
(In reply to Marcel de Rooy from comment #23)
> Koha::Email (19.11)
>
> if ( C4::Context->preference('ReturnpathDefault') ) {
> $params->{sender} ||= C4::Context->preference('ReturnpathDefault');
> }
>
> Mail::Sendmail
> If you wish to use an envelope sender address different than the From:
> address, set $mail{Sender} in your %mail hash
>
> MTAs will copy the Sender to Return-path
>
> But in the meantime master has been changed quite a bit.
> So the solution of this problem must be found in these changes?
Thx for digging this up. It's also the explanation I found so far.
The strange thing now is that we see the wrong setting in Return-Path. Instead
of using the information from Sender it appears that From is used for setting
the Return-Path.
I would suspect that this is not done by the MTA that hasn't changed then, but
there shoudl be a visible change in the Koha email format too.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #30 from Katrin Fischer --- (In reply to Jonathan Druart from comment #27) > Comment 18 shows that Koha didn't set Return-path in previous versions. Sorry, but I have to disagree. We tested the initial development in 2014 and we have just tested with 18.11 and the Return-Path was definitely set correctly. I think it might depend on MTA and how you test the email. Local testing might be limited here. How did you test? The issue we see now is also not that it's not set, but it's set to the wrong address, which is even worse. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #29 from Ulrich Kleiber --- The previous patch does not work, because it unfortunately also changes the header-from. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729
--- Comment #28 from Ulrich Kleiber ---
Maybe in Email.pm the envelope-from should be set to the return-path.
Koha 20.11
--- Email.pm2021-07-29 11:07:24.560924469 +0200
+++ Email.pm.patched2021-07-29 11:49:01.108795689 +0200
@@ -76,9 +76,6 @@
my ( $self, $params ) = @_;
my $args = {};
-$args->{from} = $params->{from} ||
C4::Context->preference('KohaAdminEmailAddress');
-Koha::Exceptions::BadParameter->throw("Invalid 'from' parameter:
".$args->{from})
-unless Email::Valid->address( -address => $args->{from}, -fqdn => 0 );
# from is mandatory
$args->{subject} = $params->{subject} // '';
@@ -117,6 +114,9 @@
);
}
+$args->{from} = $addresses->{sender} ||
C4::Context->preference('ReturnpathDefault') || $params->{from} ||
C4::Context->preference('KohaAdminEmailAddress');
+Koha::Exceptions::BadParameter->throw("Invalid 'from' parameter:
".$args->{from})
+unless Email::Valid->address( -address => $args->{from}, -fqdn => 0 );
# from is mandatory
$args->{cc} = $addresses->{cc}
if $addresses->{cc};
$args->{bcc} = $addresses->{bcc}
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #27 from Jonathan Druart --- Comment 18 shows that Koha didn't set Return-path in previous versions. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #26 from Katrin Fischer --- > Something between versions has changed to cause the change in the > Return-Path header. Maybe another side effect of the SMTP library change? That's what I was suspecting all along :) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #25 from Marcel de Rooy --- (In reply to Marcel de Rooy from comment #24) > Bug 22343 ? This one was pushed to 20.11 not backported. I think that the problem is there. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #24 from Marcel de Rooy --- Bug 22343 ? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729
--- Comment #23 from Marcel de Rooy ---
Koha::Email (19.11)
if ( C4::Context->preference('ReturnpathDefault') ) {
$params->{sender} ||= C4::Context->preference('ReturnpathDefault');
}
Mail::Sendmail
If you wish to use an envelope sender address different than the From: address,
set $mail{Sender} in your %mail hash
MTAs will copy the Sender to Return-path
But in the meantime master has been changed quite a bit.
So the solution of this problem must be found in these changes?
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Marcel de Rooy changed: What|Removed |Added CC||m.de.r...@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #22 from David Cook --- (In reply to David Cook from comment #21) > I wouldn't be surprised if this were related to Bug 26705. I mean... not the same problem/fix, but that the new library might be messing with things. I could be totally wrong though. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 David Cook changed: What|Removed |Added CC||dc...@prosentient.com.au --- Comment #21 from David Cook --- I wouldn't be surprised if this were related to Bug 26705. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #20 from Ulrich Kleiber --- Hi Jonathan, the configuration of the MTA was not changed when upgrading to 20.11. It is the same on all our Koha Servers. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #19 from Jonathan Druart --- Did you check the MTA config? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #18 from Jonathan Druart --- Tried on 18.11.x -- MESSAGE FOLLOWS -- Sender: returnpathdefa...@default.org Mime-Version: 1.0 Charset: utf8 Content-Transfer-Encoding: quoted-printable Subject: =?UTF-8?B?Q2hlY2tvdXRz?= Content-Type: text/plain; charset="UTF-8" Date: Wed, 28 Jul 2021 12:59:39 + To: e...@example.org From: r...@example.org X-Peer: 127.0.0.1 The following items have been checked out: E Street shuffle=20 Thank you for visiting Centerville.= END MESSAGE -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #17 from Katrin Fischer --- (In reply to Jonathan Druart from comment #16) > Looks like we never set the "Return-Path" from Koha and haven't found > anything related in the Mail::Sendmail library. But it worked. Just because we haven't found out why it changed yet, doesn't mean this isn't a regression. Because the behaviour change can be proved. The Return-path functionality was developed, tested and agreed on in bug 9350 in 2014. I can promise it worked when added to Koha and has worked since then at least up to 18.11 but probably in later versions as well. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #16 from Jonathan Druart --- Looks like we never set the "Return-Path" from Koha and haven't found anything related in the Mail::Sendmail library. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Katrin Fischer changed: What|Removed |Added Keywords||regression -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Katrin Fischer changed: What|Removed |Added Severity|normal |major --- Comment #15 from Katrin Fischer --- Any library relying on the return-path paramters might be likely to no longer receive emails bouncing back. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 --- Comment #14 from Katrin Fischer --- (In reply to Jonathan Druart from comment #13) > So if I understand correctly this is a new enhancement request, not a > regression? No, it's a regression and a bug. We have configuration options for the Return-Path in Koha that now are no longer used/functional. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28729] Return-path header not set in emails
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28729 Jonathan Druart changed: What|Removed |Added Severity|critical|normal Summary|Return-path header not |Return-path header not set |correctly set in emails |in emails Version|20.11 |unspecified --- Comment #13 from Jonathan Druart --- So if I understand correctly this is a new enhancement request, not a regression? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
