Re: [svn:qpsmtpd] r740 - in trunk: . lib/Qpsmtpd plugins

On May 17, 2007, at 15:30, Matt Sergeant wrote:

> On 17-May-07, at 6:27 PM, Ask Bj=F8rn Hansen wrote:
>
>>
>> On May 17, 2007, at 15:16, [email protected] wrote:
>>
>> +my $has_ipv6 =3D Qpsmtpd::TcpServer::has_ipv6;
>>
>> Does that work with the Apache and the async server?
>
> Sort of :-)
>
> It works in the sense that all has_ipv6() does is check the perl =20
> IPv6 libraries load. But -async (and I think Apache::*) has no =20
> support for ipv6.

Well, it makes the tests break.  :-)

>> Also - I'm not entirely sure how it should work, but it seems like =20=

>> there are some opportunities for putting some of the "deal with IP =20=

>> addresses" stuff in as helper methods in Qpsmtpd::Plugin (or =20
>> another "helper base class").
>
> No, I think the way it's done is specific to TcpServer, mostly.

(Just to be clear) I was thinking the stuff that went into the =20
plugins -- it's nasty to have it be TcpServer specific (for one =20
thing) and it seems like some of it should be general enough to be in =20=

the API somehow.    (Not important now, but worth keeping in mind =20
before we add more IPv6 support).


  - ask

--=20
http://develooper.com/ - http://askask.com/