]> Sergey Matveev's repositories - public-inbox.git/commitdiff
www: inbox: favor "pop3://" over "pop://"
authorEric Wong <e@80x24.org>
Thu, 11 Aug 2022 20:00:20 +0000 (20:00 +0000)
committerEric Wong <e@80x24.org>
Thu, 11 Aug 2022 20:13:54 +0000 (20:13 +0000)
curl only supports "pop3://" and "pop3s://", despite RFC 2384
existing for "pop://".  AFAIK, there's no RFCs for "pop3://"
and "pop3s://", but please let us know if there are.

In any case, real-world cases like curl are more relevant.

lib/PublicInbox/Inbox.pm

index 3f70e69d53d3229e4a5a78bde97ce3ff1b31c571..1f7f982da783fee8e847e98294828ace331832ab 100644 (file)
@@ -285,9 +285,9 @@ sub pop3_url {
                my $group = $self->{newsgroup};
                my @urls;
                ($ps && $group) and
-                       @urls = map { m!\Apops?://! ? $_ : "pop://$_" } @$ps;
+                       @urls = map { m!\Apop3?s?://! ? $_ : "pop3://$_" } @$ps;
                if (my $mi = $self->{'pop3mirror'}) {
-                       my @m = map { m!\Apops?://! ? $_ : "pop://$_" } @$mi;
+                       my @m = map { m!\Apop3?s?://! ? $_ : "pop3://$_" } @$mi;
                        my %seen; # List::Util::uniq requires Perl 5.26+
                        @urls = grep { !$seen{$_}++ } (@urls, @m);
                }