Skip Menu |

This queue is for tickets about the IO-Socket-SSL CPAN distribution.

Report information
The Basics
Id: 32960
Status: resolved
Priority: 0/
Queue: IO-Socket-SSL

People
Owner: Nobody in particular
Requestors: tom@eborcom.com (no email address)
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 1.13
Fixed in: (no value)



Subject: Inaccurate Debugging Documentation
The "Debugging" section of this module's documentation describes 4 debugging levels that provide different amounts of debugging information. The module fails to support these levels, instead it either switches debugging on or off (see sub DEBUG). Furthermore, the module contains many useful but commented out debugging messages: % grep '#DEBUG' IO/Socket/SSL.pm 23 It would be very helpful to uncomment the debugging messages, then modify the calls to DEBUG and its subroutine to support levels as per the documentation. Alternatively, the documentation should reflect the module's current behaviour. Thanks, Tom Hukins
On Mi. 06. Feb. 2008, 03:57:40, tom@eborcom.com wrote: Show quoted text
> The "Debugging" section of this module's documentation describes 4 > debugging levels that provide different amounts of debugging information. > > The module fails to support these levels, instead it either switches > debugging on or off (see sub DEBUG).
The variable IO::Socket::SSL is an alias for Net::SSLeay::trace so setting this variable also sets the Net::SSLeay trace level which distinguisches between various debug levels. I've added documentation for version 1.14 to clarify this. Show quoted text
> > Furthermore, the module contains many useful but commented out debugging > messages: > % grep '#DEBUG' IO/Socket/SSL.pm > 23 > > It would be very helpful to uncomment the debugging messages, then > modify the calls to DEBUG and its subroutine to support levels as per > the documentation. Alternatively, the documentation should reflect the > module's current behaviour.
I've enabled some more debugging in version 1.14 and documented the behavior. Thanks, Steffen