[PATCH] PublicInbox::Inbox.pm: Default unset address to a one element array

[PATCH] PublicInbox::Inbox.pm: Default unset address to a one element array

[Eric W. Biederman]

PublicInbox::Config.pm::_fill() assumes that address is an array.
Therefore when handling an unset address use an array containing a
single string, instead of a single string.

Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
---

I accidentially created a public inbox without an address at some
point and had some nasty errors because the default value
we game to address is not an array.

 lib/PublicInbox/Inbox.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/PublicInbox/Inbox.pm b/lib/PublicInbox/Inbox.pm
index 617b692b14ca..002b980f4057 100644
--- a/lib/PublicInbox/Inbox.pm
+++ b/lib/PublicInbox/Inbox.pm
@@ -102,7 +102,7 @@ sub _set_limiter ($$$) {
 
 sub new {
 	my ($class, $opts) = @_;
-	my $v = $opts->{address} ||= 'public-inbox@example.com';
+	my $v = $opts->{address} ||= [ 'public-inbox@example.com' ];
 	my $p = $opts->{-primary_address} = ref($v) eq 'ARRAY' ? $v->[0] : $v;
 	$opts->{domain} = ($p =~ /\@(\S+)\z/) ? $1 : 'localhost';
 	my $pi_config = delete $opts->{-pi_config};
-- 
2.20.1

Re: [PATCH] PublicInbox::Inbox.pm: Default unset address to a one element array

[Eric Wong]

"Eric W. Biederman" <ebiederm@xmission.com> wrote:
> 
> PublicInbox::Config.pm::_fill() assumes that address is an array.
> Therefore when handling an unset address use an array containing a
> single string, instead of a single string.
> 
> Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>

Thanks, pushed as 0a556c99c0db5160f43931422a563c6094b64ea2

> I accidentially created a public inbox without an address at some
> point and had some nasty errors because the default value
> we game to address is not an array.

Yeah, I've been thinking about making it more suitable for
personal mail, and the address field requirement is a bit
annoying.

Though I suppose it could be useful for making search queries
easier by expanding queries, using "$self" as a placeholder
to expand to all defined addresses for folks with multiple
email addresses... but maybe .mailmap is better for that.