Patch removed from consideration [WAS: Re: Patch consideration question for crypt in libsasl2]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Oct 15, 2006 at 01:02:08AM -0200, Henrique de Moraes Holschuh wrote:
> On Sat, 14 Oct 2006, Roberto C. Sanchez wrote:
> > A patch [0] was proposed a while back on the cyrus-sasl list to allow
> > crypt in libsasl2.  The original message [1] is also available.  Anyhow,
> 
> We should not accept that patch *ever* in any other format than a "optional,
> *disabled by default* thing you should enable only if you know what you are
> doing".  And I would not include it even in that form.  
> 
> It breaks auxprop plugins, which is a fundamental way of how Cyrus SASL
> works.  It requires disabling globally some auth methods [that require the
> cleartext password to generate challenges] when the feature is enabled too,
> if the patch doesn't do this, please reject it without futher consideration.
> 
> > I'd like to hear people's opinions on this.  My vote (for the Debian
> > people) is that if upstream says that they don't want the patch, that we
> > close the bug and be done with it.
> 
> Upstream didn't want the patch.  They may change their mind (they have done
> so with the 8-bit stuff in cyrus imapd headers, which goes against *all*
> RFCs), though.
> 
OK.  I thought it would cause at least some breakage, though I had no
idea of the scope.  Given Henrique's comments, I have decided to close
the bug.  Upstream can disregard it from consideration.  Unless someone
has an exceptionally compelling case for why this should be different, I
think this is how it should stay.

Regards,

-Roberto

-- 
Roberto C. Sanchez
http://people.connexer.com/~roberto
http://www.connexer.com

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Info Cyrus]     [Squirrel Mail]     [Linux Media]     [Yosemite News]     [gtk]     [KDE]     [Gimp on Windows]     [Steve's Art]

  Powered by Linux