Open schellj opened 1 year ago
There was discussion about mail rewrite behavior in relation to the allow_resend_from_re config (https://github.com/biviosoftware/perl-Bivio/pull/54#discussion_r1013456870) that resulted in commit https://github.com/biviosoftware/perl-Bivio/commit/6e5b2c66401e51c53c0a2ba25f9b4f1661e85782, which caused some tests to fail unexpectedly (Mail/t/Outgoing2.bunit, PetShop/Test/t/mail.btest, PetShop/Test/t/rewrite-from-domain.btest). It was assumed previously that allow_resend_from_re was only relevant to tests, but it is actually used in production also. That commit was reverted in #55.
allow_resend_from_re
Mail/t/Outgoing2.bunit
PetShop/Test/t/mail.btest
PetShop/Test/t/rewrite-from-domain.btest
Is the current behavior correct and/or desirable?
There was discussion about mail rewrite behavior in relation to the
allow_resend_from_re
config (https://github.com/biviosoftware/perl-Bivio/pull/54#discussion_r1013456870) that resulted in commit https://github.com/biviosoftware/perl-Bivio/commit/6e5b2c66401e51c53c0a2ba25f9b4f1661e85782, which caused some tests to fail unexpectedly (Mail/t/Outgoing2.bunit
,PetShop/Test/t/mail.btest
,PetShop/Test/t/rewrite-from-domain.btest
). It was assumed previously thatallow_resend_from_re
was only relevant to tests, but it is actually used in production also. That commit was reverted in #55.Is the current behavior correct and/or desirable?