Patch removed from consideration [WAS: Re: Patch consideration
question for crypt in libsasl2]
Roberto C. Sanchez
roberto at connexer.com
Sun Oct 15 06:59:14 EDT 2006
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  was proposed a while back on the cyrus-sasl list to allow
> > crypt in libsasl2. The original message  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.
Roberto C. Sanchez
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
Url : https://lists.andrew.cmu.edu/mailman/private/cyrus-sasl/attachments/20061015/31553fe2/attachment.bin
More information about the Cyrus-sasl