]> Sergey Matveev's repositories - public-inbox.git/commit
nntpd: move {newsgroup} name check to config
authorEric Wong <e@80x24.org>
Tue, 1 Dec 2020 01:35:46 +0000 (01:35 +0000)
committerEric Wong <e@80x24.org>
Tue, 1 Dec 2020 10:19:33 +0000 (10:19 +0000)
commitcb75a7aeb9fd530b1816b760c8c7f6cfb428c8e3
treea892c26517aced341d8ea15a33f984da3497bb3a
parent4cc4ad59287a4d040a993267065739f416da7397
nntpd: move {newsgroup} name check to config

With 50K newsgroups in the config file, this doesn't slow down
`PublicInbox::Config->new->fill_all' any measurable amount on
my busy old workstation.

This should prevent invalid newsgroup names from getting into
into extindex and catch user errors sooner, rather than later.

v2:
  - delete {newsgroup} if invalid to avoid ->nntp_url link
  - simplify -imapd and explain remaining check
lib/PublicInbox/Config.pm
lib/PublicInbox/IMAPD.pm
lib/PublicInbox/NNTPD.pm