Skip Menu |

This queue is for tickets about the Net-ManageSieve CPAN distribution.

Report information
The Basics
Id: 58192
Status: resolved
Priority: 0/
Queue: Net-ManageSieve

People
Owner: SKA [...] cpan.org
Requestors: Rudy.Gevaert [...] UGent.be
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in:
  • 0.03
  • 0.04
  • 0.05
  • 0.06
  • 0.07
  • 0.08
  • 0.09
Fixed in: 0.10



Subject: support for authorization
Date: Mon, 07 Jun 2010 12:57:16 +0200
To: bug-Net-ManageSieve [...] rt.cpan.org
From: Rudy Gevaert <Rudy.Gevaert [...] UGent.be>
Dear maintainer, It would be nice to see your module implement authorization under sasl plain. Then it is possible to login as the cyrus user and authorize as different user. For now the module takes user the same as authname. Changing: $sasl = Authen::SASL->new(mechanism=> "".$mech, callback => { user => 'cyrus', pass => $password, password => $password, authname => $username, } ); provides the wanted result. It would be nice if this could be set with a parameter. Would you consider this please? Thanks! Rudy
On Mon Jun 07 07:01:43 2010, Rudy.Gevaert@UGent.be wrote: Dear Rudy, Show quoted text
> It would be nice to see your module implement authorization under
sasl plain. [...] Show quoted text
> It would be nice if this could be set with a parameter.
Please try attached patch along with the modifaction of the POD. It is backward compatible that way. Regards, Steffen
Subject: authname.patch.gz
Download authname.patch.gz
application/x-gzip 1006b

Message body not shown because it is not plain text.

Subject: Re: [rt.cpan.org #58192] support for authorization
Date: Mon, 21 Jun 2010 15:41:43 +0200
To: bug-Net-ManageSieve [...] rt.cpan.org
From: Rudy Gevaert <Rudy.Gevaert [...] UGent.be>
On 06/17/2010 03:41 PM, Steffen Kaiser via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=58192> > > On Mon Jun 07 07:01:43 2010, Rudy.Gevaert@UGent.be wrote: > Dear Rudy,
>> It would be nice to see your module implement authorization under
> sasl plain. > [...]
>> It would be nice if this could be set with a parameter.
> > Please try attached patch along with the modifaction of the POD. > > It is backward compatible that way. > > Regards, Steffen
Hello Steffen, I have tested your patch and it works! (But you already knew that of course ;) Thanks for considering and applying!! Rudy
fixed in v0.10