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.
