[PATCH] Import.pm: Deal with potentially missing From and Sender headers

[PATCH] Import.pm: Deal with potentially missing From and Sender headers

[Eric W. Biederman]

Use ||= '' to ensure that if the From or Sender header is not present
the code sees an empty string and instead of undefined.

I had some email messages with a From field without an @ (because the
sender was local) and without a Sender which were causing errors when
imported.  I think this was bad enough that the email messages were
failing to be imported.

Signed-off-by: Eric Biederamn <ebiederm@xmission.com>
---
 lib/PublicInbox/Import.pm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/lib/PublicInbox/Import.pm b/lib/PublicInbox/Import.pm
index f320c58c6b57..4e3b4c551797 100644
--- a/lib/PublicInbox/Import.pm
+++ b/lib/PublicInbox/Import.pm
@@ -278,10 +278,12 @@ sub extract_author_info ($) {
 
 	my $sender = '';
 	my $from = $mime->header('From');
+	$from ||= '';
 	my ($email) = PublicInbox::Address::emails($from);
 	my ($name) = PublicInbox::Address::names($from);
 	if (!defined($name) || !defined($email)) {
 		$sender = $mime->header('Sender');
+		$sender ||= '';
 		if (!defined($name)) {
 			($name) = PublicInbox::Address::names($sender);
 		}
-- 
2.17.1

Re: [PATCH] Import.pm: Deal with potentially missing From and Sender headers

[Eric Wong]

"Eric W. Biederman" <ebiederm@xmission.com> wrote:
> 
> Use ||= '' to ensure that if the From or Sender header is not present
> the code sees an empty string and instead of undefined.
> 
> I had some email messages with a From field without an @ (because the
> sender was local) and without a Sender which were causing errors when
> imported.  I think this was bad enough that the email messages were
> failing to be imported.

Thanks, pushed to master as ce5af2029b6bc703f3d8cf7bb5c68732c41f210e.

I assume this is from testing against personal mail?
I haven't tried that, yet :x

Re: [PATCH] Import.pm: Deal with potentially missing From and Sender headers

[Eric W. Biederman]

Eric Wong <e@80x24.org> writes:

> "Eric W. Biederman" <ebiederm@xmission.com> wrote:
>> 
>> Use ||= '' to ensure that if the From or Sender header is not present
>> the code sees an empty string and instead of undefined.
>> 
>> I had some email messages with a From field without an @ (because the
>> sender was local) and without a Sender which were causing errors when
>> imported.  I think this was bad enough that the email messages were
>> failing to be imported.
>
> Thanks, pushed to master as ce5af2029b6bc703f3d8cf7bb5c68732c41f210e.
>
> I assume this is from testing against personal mail?
> I haven't tried that, yet :x

It is.  So far just importing my personal mail.

Eric