Avelsieve Commit 851 and Authentication Method Case
by: Craig Shelley <craig.NOSP@MMER.microtron.org.uk>, @ Tue, 05 Jan 2010 00:33:07 +0200

Hello,

Avelsive commit 851 "Ignore case for Sieve capabilities list" also
causes authentication methods to also be converted to lower case eg
'plain' instead of 'PLAIN'
This makes authentication impossible over TLS with my setup.

Reverting commit 851 in addition to setting
$this->broken_tls =3D true;
As described here;
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=3D547648
allows Avelsieve 1.9.9 to work with debian cyrus 2.2.13-14+lenny3


Regards,

Craig Shelley



Commit 851:
--- avelsieve/main_plugin/trunk/include/managesieve.lib.php
2007/04/11
10:36:59 850
+++ avelsieve/main_plugin/trunk/include/managesieve.lib.php
2007/04/11 10:41:42 851
@@ -2,7 +2,7 @@
/**
* sieve-php.lib.php
*
- * $Id: managesieve.lib.php,v 1.11 2007/01/17 13:46:10 avel Exp $=20
+ * $Id: managesieve.lib.php,v 1.12 2007/04/11 10:41:42 avel Exp $=20
*
* Copyright 2001-2003 Dan Ellis
*
@@ -867,10 +867,10 @@
$this->modules =3D split(' ', $this->item[1]);
if(is_array($this->modules)){
foreach($this->modules as $m) {
- $this->capabilities[$cap_type][$m]=3Dtrue;
+
$this->capabilities[$cap_type][strtolower($m)]=3Dtrue;
}
} elseif(is_string($this->modules)) {
- $this->capabilites[$cap_type][$this->modules]=3Dtrue;
+
$this->capabilites[$cap_type][strtolower($this->modules)]=3Dtrue;
}
} else {=20
$this->capabilities["unknown"][]=3D$this->line;

Attachments: