Just having "limiter" in the prefix may confuse it with something else. Use the full prefix to avoid this confusion. --- lib/PublicInbox/Config.pm | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/lib/PublicInbox/Config.pm b/lib/PublicInbox/Config.pm index 5adcd0c..8d66cf8 100644 --- a/lib/PublicInbox/Config.pm +++ b/lib/PublicInbox/Config.pm @@ -90,8 +90,12 @@ sub limiter { my ($self, $name) = @_; $self->{-limiters}->{$name} ||= do { require PublicInbox::Qspawn; - my $key = "limiter.$name.max"; - PublicInbox::Qspawn::Limiter->new($self->{$key}); + my $max; + # XXX "limiter.<name>.max" was a historical mistake + foreach my $pfx (qw(publicinboxlimiter limiter)) { + $max ||= $self->{"$pfx.$name.max"}; + } + PublicInbox::Qspawn::Limiter->new($max); }; } -- EW