Hi,

On Fri, 5 May 2006, [EMAIL PROTECTED] wrote:

I started NTLM integration few weeks ago and due to its very
experimental nature I decided to have a configure option for disabling
it. I'd be for additional su_configure.h define, because NTLM support
might and will be broken several times.

it would be best to avoid any unnecessary feature-specific parts in the public headers. If we have these, then a simple pkg-config check for sofia-sip won't be enough anymore for certain client.

I'd suggest hiding the dependency. I.e. remove dependency between auth_client.h and auth_ntlm.h, and instead include auth_ntlm.h from auth_client.c. Now you don't need to install auth_ntlm.h at all, and auth_ntlm.h can be hidden until it is ready to be part of the stable public API.

You could still have a NTLM check via the 'features' module, with which clients could check for NTLM support during runtime.

How does this sound?

--
 under work: Sofia-SIP at http://sofia-sip.sf.net


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Sofia-sip-devel mailing list
Sofia-sip-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sofia-sip-devel

Reply via email to