Re: white-listing in hosts_allow plugin

Robin Bowes wrote:
> 
> On 09/12/10 13:10, Robin Bowes wrote:
> > Hi,
> > 
> > I'd like to add the capability to white-list in the hosts_allow plugin.
> > Specifically, we have a relay that we expect to get multiple connections
> > from so I don't want to restrict connections from that IP.
> > 
> > I was looking at the hosts_allow code, trying to work out the best way
> > to implement this without duplication. Then it hit me that if we just
> > reverse the order of the tests, we can use the hosts_allow mechanism to
> > white-list hosts.
> 
> Actually, the more I think about this, the more I think that the
> connections-per-ip code should be in a different plugin to the
> hosts_allow code, and should have a separate config file which would
> allow per-host max connection limits.
> 
> Is that something you'd be interested in merging back with HEAD?

Most definitely.  I could see using this to set different limits for
internal and external hosts.

IIRC, the connections-per-ip code isn't in a plugin right now -- it's
part of the main binary -- because it requires some sort of shared
state which we don't have for plugins.