Bug #74658 for Net-SSLeay: Which openssl versions does Net::SSLeay support?

Fri Feb 03 08:18:39 2012 kmx [...] volny.cz - Ticket created

Subject:	Which openssl versions does Net::SSLeay support?
Date:	Fri, 03 Feb 2012 14:18:25 +0100
To:	bug-Net-SSLeay [...] rt.cpan.org
From:	kmx <kmx [...] volny.cz>

Makefile.PL seems to require at least openssl 0.9.3 However according my testing Net::SSLeay does not build correctly even with 0.9.8 (without any letter). Of course it could be easily fixed with some #ifdefs The question is what is the minimal required version? (once we have the answer we can put proper #ifdefs into SSLeay.xs). -- kmx

Fri Feb 03 17:13:41 2012 mikem [...] open.com.au - Correspondence added

Subject:	Re: [rt.cpan.org #74658] Which openssl versions does Net::SSLeay support?
Date:	Sat, 04 Feb 2012 08:13:29 +1000
To:	bug-Net-SSLeay [...] rt.cpan.org
From:	Mike McCauley <mikem [...] open.com.au>

Hi, On Friday, February 03, 2012 08:18:40 AM you wrote: Show quoted text

> Fri Feb 03 08:18:39 2012: Request 74658 was acted upon. > Transaction: Ticket created by kmx@volny.cz > Queue: Net-SSLeay > Subject: Which openssl versions does Net::SSLeay support? > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: kmx@volny.cz > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=74658 > > > > Makefile.PL seems to require at least openssl 0.9.3 > > However according my testing Net::SSLeay does not build correctly even > with 0.9.8 (without any letter). > > Of course it could be easily fixed with some #ifdefs > > The question is what is the minimal required version? (once we have the > answer we can put proper #ifdefs into SSLeay.xs).

The history of this is from before my time, but I think we should continue to support versions that are currently supported. If we cant do that, then I think users deserve some notice? Cheers. Show quoted text

> > -- > kmx

-- Mike McCauley mikem@open.com.au Open System Consultants Pty. Ltd 9 Bulbul Place Currumbin Waters QLD 4223 Australia http://www.open.com.au Phone +61 7 5598-7474 Fax +61 7 5598-7070 Radiator: the most portable, flexible and configurable RADIUS server anywhere. SQL, proxy, DBM, files, LDAP, NIS+, password, NT, Emerald, Platypus, Freeside, TACACS+, PAM, external, Active Directory, EAP, TLS, TTLS, PEAP, TNC, WiMAX, RSA, Vasco, Yubikey, MOTP, HOTP, TOTP, DIAMETER etc. Full source on Unix, Windows, MacOSX, Solaris, VMS, NetWare etc.

Fri Feb 03 17:13:42 2012 The RT System itself - Status changed from 'new' to 'open'

Sun Feb 05 15:46:18 2012 kmx [...] volny.cz - Correspondence added

Subject:	Re: [rt.cpan.org #74658] Which openssl versions does Net::SSLeay support?
Date:	Sun, 05 Feb 2012 21:46:00 +0100
To:	bug-Net-SSLeay [...] rt.cpan.org
From:	kmx <kmx [...] volny.cz>

Show quoted text

> The history of this is from before my time, but I think we should continue to > support versions that are currently supported. If we cant do that, then I > think users deserve some notice? >

I have done some research and prepare a patch. The most important conclusions: - before my patch 0.9.8a+ was ok (build without warnings/errors, all tests pass) - fixing support for 0.9.8 (without letter) was easy but I needed to add a function to get openssl version (to selectively skip some tests) - fixing support for 0.9.7 was some more work but in the end it builds OK and all tests pass - fixing support for 0.9.6 is kind of "not-guaranteed-to-work" - it builds OK but it crashes during tests on my Win32 box (as it is so old and so far out of my interest I did not invest more time into this) - fixing support for 0.9.5 or earlier I have not even tried (0.9.5 series end by 0.9.5a - Apr 3 2000, IMHO not worth any effort) The enclosed patch contains: - new or updated #ifdefs in SSLeay.xs - 2 new version related functions: Net::SSLeay::SSLeay(); Net::SSLeay::SSLeay_version(); - tests + doc for the above mentioned 2 functions One extra thing not related to this RT I have included in my patch is a doc enhancement for some low level API (I have chosen RAND_ stuff). I it intended as an idea how I think it would be nice to have documented openssl low level API functions - if you do not like it simply revert my change; if you like it I can try to update doc also for other low level api (X509_* CTX_* ...). -- kmx

Message body is not shown because sender requested not to inline it.

Sun Feb 05 17:40:49 2012 mikem [...] open.com.au - Correspondence added

Subject:	Re: [rt.cpan.org #74658] Which openssl versions does Net::SSLeay support?
Date:	Mon, 06 Feb 2012 08:40:39 +1000
To:	bug-Net-SSLeay [...] rt.cpan.org
From:	Mike McCauley <mikem [...] open.com.au>

Hi, Nice work. Patched and committed to 291 Cheers, On Sunday, February 05, 2012 03:46:20 PM you wrote: Show quoted text

> Queue: Net-SSLeay > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=74658 > >

> > The history of this is from before my time, but I think we should > > continue to support versions that are currently supported. If we cant > > do that, then I think users deserve some notice?

> > I have done some research and prepare a patch. > > The most important conclusions: > - before my patch 0.9.8a+ was ok (build without warnings/errors, all > tests pass) > - fixing support for 0.9.8 (without letter) was easy but I needed to add > a function to get openssl version (to selectively skip some tests) > - fixing support for 0.9.7 was some more work but in the end it builds > OK and all tests pass > - fixing support for 0.9.6 is kind of "not-guaranteed-to-work" - it > builds OK but it crashes during tests on my Win32 box (as it is so old > and so far out of my interest I did not invest more time into this) > - fixing support for 0.9.5 or earlier I have not even tried (0.9.5 > series end by 0.9.5a - Apr 3 2000, IMHO not worth any effort) > > The enclosed patch contains: > - new or updated #ifdefs in SSLeay.xs > - 2 new version related functions: > Net::SSLeay::SSLeay(); > Net::SSLeay::SSLeay_version(); > - tests + doc for the above mentioned 2 functions > > One extra thing not related to this RT I have included in my patch is a > doc enhancement for some low level API (I have chosen RAND_ stuff). I it > intended as an idea how I think it would be nice to have documented > openssl low level API functions - if you do not like it simply revert my > change; if you like it I can try to update doc also for other low level > api (X509_* CTX_* ...). > > -- > kmx

-- Mike McCauley mikem@open.com.au Open System Consultants Pty. Ltd 9 Bulbul Place Currumbin Waters QLD 4223 Australia http://www.open.com.au Phone +61 7 5598-7474 Fax +61 7 5598-7070 Radiator: the most portable, flexible and configurable RADIUS server anywhere. SQL, proxy, DBM, files, LDAP, NIS+, password, NT, Emerald, Platypus, Freeside, TACACS+, PAM, external, Active Directory, EAP, TLS, TTLS, PEAP, TNC, WiMAX, RSA, Vasco, Yubikey, MOTP, HOTP, TOTP, DIAMETER etc. Full source on Unix, Windows, MacOSX, Solaris, VMS, NetWare etc.

Sun Feb 05 17:41:24 2012 MIKEM [...] cpan.org - TimeWorked changed from (no value) to '10'

Sun Feb 05 17:41:24 2012 MIKEM [...] cpan.org - Status changed from 'open' to 'resolved'

Sun Feb 05 17:41:25 2012 MIKEM [...] cpan.org - Given to MIKEM