Skip Menu |

This queue is for tickets about the Mail-IMAPClient CPAN distribution.

Report information
The Basics
Id: 53998
Status: resolved
Priority: 0/
Queue: Mail-IMAPClient

People
Owner: Nobody in particular
Requestors: dragoslav.mlakar [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: (no value)
Fixed in: 3.23



Subject: NTLM not working; tested on cygwin perl <->Exchange 2003 with NTLM IMAP
#NTLM not working because phase2 create response without challenge salt... #buggy: elsif ( $scheme eq 'NTLM' ) { $response ||= sub { my ( $code, $client ) = @_; require Authen::NTLM; Authen::NTLM::ntlm_user( $self->User ); Authen::NTLM::ntlm_password( $self->Password ); Authen::NTLM::ntlm_domain( $self->Domain ) if $self->Domain; Authen::NTLM::ntlm(); }; } #working,tested: elsif ( $scheme eq 'NTLM' ) { $response ||= sub { my ( $code, $client ) = @_; require Authen::NTLM; Authen::NTLM::ntlm_user( $self->User ); Authen::NTLM::ntlm_password( $self->Password ); Authen::NTLM::ntlm_domain( $self->Domain ) if $self->Domain; Authen::NTLM::ntlm($code); }; }
From: dragoslav.mlakar [...] gmail.com
sorry it's not fixed yet... ;)
On Tue Jan 26 01:40:35 2010, justincase wrote: Show quoted text
> sorry it's not fixed yet... ;)
Can you elaborate on this statement? Are you saying the suggested fix does not work?
From: dragoslav.mlakar [...] gmail.com
fix is working I just meant that I state in report that is fixed in version ... Fixed in: 3.22 On Tue Jan 26 11:10:43 2010, PLOBBES wrote: Show quoted text
> On Tue Jan 26 01:40:35 2010, justincase wrote:
> > sorry it's not fixed yet... ;)
> > Can you elaborate on this statement? Are you saying the suggested fix > does not work?
Thanks for the bug report. Below is what the code that will be in the next release. Note the use of $client now instead of $self ... who knows how long that has been broken. And of course, use of challenge ($code) as you suggested in the call to ntlm(): elsif ( $scheme eq 'NTLM' ) { $response ||= sub { my ( $code, $client ) = @_; require Authen::NTLM; Authen::NTLM::ntlm_user( $client->User ); Authen::NTLM::ntlm_password( $client->Password ); Authen::NTLM::ntlm_domain( $client->Domain ) if $client- Show quoted text
>Domain;
Authen::NTLM::ntlm($code); }; } I'll close this bug after 3.23 is released.
The patch was included with the 3.23 release which went out last week. Please feel free to reopen if you get a chance to test and find any problems!