user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
* [PATCH] www: uninitialized vars due to extindex lacking address
@ 2021-08-16  5:30 Eric Wong
  0 siblings, 0 replies; only message in thread
From: Eric Wong @ 2021-08-16  5:30 UTC (permalink / raw)
  To: meta

Some messages have From/To/Cc headers munged to be unparseable to
Email::Address::XS, the fallback is to use the default inbox address.
-extindex do not have an address on their own, so just fall back to
using 'unknown@example.com' for now.

An example of such a message:
https://yhbt.net/lore/all/20201002154535.28412-1-fw@strlen.de/
---
 lib/PublicInbox/ExtSearch.pm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/PublicInbox/ExtSearch.pm b/lib/PublicInbox/ExtSearch.pm
index c2cfc338..0b480c7e 100644
--- a/lib/PublicInbox/ExtSearch.pm
+++ b/lib/PublicInbox/ExtSearch.pm
@@ -19,6 +19,7 @@ sub new {
 	my ($class, $topdir) = @_;
 	bless {
 		topdir => $topdir,
+		-primary_address => 'unknown@example.com',
 		# xpfx => 'ei15'
 		xpfx => "$topdir/ei".PublicInbox::Search::SCHEMA_VERSION
 	}, $class;

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2021-08-16  5:30 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-16  5:30 [PATCH] www: uninitialized vars due to extindex lacking address Eric Wong

Code repositories for project(s) associated with this public inbox

	https://80x24.org/public-inbox.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).