about summary refs log tree commit homepage
path: root/lib/PublicInbox/NetReader.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2021-06-12 00:10:45 +0000
committerEric Wong <e@80x24.org>2021-06-13 17:39:38 +0000
commitd1dbf32d5b747deb4610f98b0befdb8b6a32243a (patch)
treec427a713ea85c5ade88f5718a9cf82bba2f9322d /lib/PublicInbox/NetReader.pm
parentd00c4b6d395259ed35900bf52dffc2db3a436c6a (diff)
downloadpublic-inbox-d1dbf32d5b747deb4610f98b0befdb8b6a32243a.tar.gz
This fixes cases when users specify an IMAP or NNTP URL
with standard port numbers explicitly.

In other words, this allows users to use
"lei ls-mail-source nntps://public-inbox.org:563/" and
"lei ls-mail-source imaps://public-inbox.org:993/"
without hitting "BUG:" errors.
Diffstat (limited to 'lib/PublicInbox/NetReader.pm')
-rw-r--r--lib/PublicInbox/NetReader.pm2
1 files changed, 2 insertions, 0 deletions
diff --git a/lib/PublicInbox/NetReader.pm b/lib/PublicInbox/NetReader.pm
index bcc6cbf0..30784199 100644
--- a/lib/PublicInbox/NetReader.pm
+++ b/lib/PublicInbox/NetReader.pm
@@ -371,8 +371,10 @@ sub add_url {
         my ($self, $arg, $ls_ok) = @_;
         my $uri;
         if ($uri = imap_uri($arg, $ls_ok)) {
+                $_[1] = $$uri; # canonicalized
                 push @{$self->{imap_order}}, $uri;
         } elsif ($uri = nntp_uri($arg, $ls_ok)) {
+                $_[1] = $$uri; # canonicalized
                 push @{$self->{nntp_order}}, $uri;
         } else {
                 push @{$self->{unsupported_url}}, $arg;