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


Reply via email to