Changeset 3801 in subversion

Timestamp:

Jul 1, 2010 6:18:44 AM (3 years ago)

Author:

alec

Message:

• Net_Sieve-1.3.0 with LOGIN fix (#1486816)

Location:

trunk/plugins/managesieve

Files:

• Changelog (modified) (1 diff)
• lib/Net/Sieve.php (modified) (7 diffs)

trunk/plugins/managesieve/Changelog

@@ -1 @@
-- Update Net_Sieve to version 1.2.2
+- Update Net_Sieve to version 1.3.0 (fixes LOGIN athentication)
 
 * version 2.6 [2010-06-03]

trunk/plugins/managesieve/lib/Net/Sieve.php

@@ -40 +40 @@
  * @copyright 2006-2008 Anish Mistry
  * @license   http://www.opensource.org/licenses/bsd-license.php BSD
- * @version   SVN: $Id: Sieve.php 300419 2010-06-13 12:10:10Z yunosh $
+ * @version   SVN: $Id: Sieve.php 300898 2010-07-01 09:49:02Z yunosh $
  * @link      http://pear.php.net/package/Net_Sieve
  */
@@ -84 +84 @@
  * @copyright 2006-2008 Anish Mistry
  * @license   http://www.opensource.org/licenses/bsd-license.php BSD
- * @version   Release: 1.2.2
+ * @version   Release: 1.3.0
  * @link      http://pear.php.net/package/Net_Sieve
  * @link      http://www.ietf.org/rfc/rfc3028.txt RFC 3028 (Sieve: A Mail
@@ -205 +205 @@
      * @param array   $options    Additional options for
      *                            stream_context_create().
+     * @param mixed   $handler    A callback handler for the debug output.
      */
     function Net_Sieve($user = null, $pass  = null, $host = 'localhost',
-        $port = 2000, $logintype = '', $euser = '', $debug = false,
-        $bypassAuth = false, $useTLS = true, $options = null
-    ) {
+                       $port = 2000, $logintype = '', $euser = '',
+                       $debug = false, $bypassAuth = false, $useTLS = true,
+                       $options = null, $handler = null)
+    {
         $this->_state             = NET_SIEVE_STATE_DISCONNECTED;
         $this->_data['user']      = $user;
@@ -218 +220 @@
         $this->_data['euser']     = $euser;
         $this->_sock              = new Net_Socket();
-        $this->_debug             = $debug;
         $this->_bypassAuth        = $bypassAuth;
         $this->_useTLS            = $useTLS;
         $this->_options           = $options;
+        $this->setDebug($debug, $handler);
 
         /* Try to include the Auth_SASL package.  If the package is not
@@ -636 +638 @@
             return $result;
         }
-        if (PEAR::isError($result = $this->_doCmd('"' . base64_encode($user) . '"'))) {
+        if (PEAR::isError($result = $this->_doCmd('"' . base64_encode($user) . '"', true))) {
             return $result;
         }
-        return $this->_doCmd('"' . base64_encode($pass) . '"');
+        return $this->_doCmd('"' . base64_encode($pass) . '"', true);
     }
 
@@ -1100 +1102 @@
             return PEAR::raiseError('This server doesn\'t support any authentication methods. SASL problem?');
         }
-
-        $serverMethods = $this->_capability['sasl'];
+        if (!$this->_capability['sasl']) {
+            return PEAR::raiseError('This server doesn\'t support any authentication methods.');
+        }
 
         if ($userMethod) {
-            $methods = array($userMethod);
-        } else {
-            $methods = $this->supportedAuthMethods;
-        }
-
-        if (!$methods || !$serverMethods) {
+            if (in_array($userMethod, $this->_capability['sasl'])) {
+                return $userMethod;
+            }
             return PEAR::raiseError(
-                'This server doesn\'t support any authentication methods.'
-            );
-        }
-
-        foreach ($methods as $method) {
-            if (in_array($method, $serverMethods)) {
+                sprintf('No supported authentication method found. The server supports these methods: %s, but we want to use: %s',
+                        implode(', ', $this->_capability['sasl']),
+                        $userMethod));
+        }
+
+        foreach ($this->supportedAuthMethods as $method) {
+            if (in_array($method, $this->_capability['sasl'])) {
                 return $method;
             }
@@ -1122 +1123 @@
 
         return PEAR::raiseError(
-            'No supported authentication method found. The server supports these methods: '
-            . implode(',', $serverMethods)
-            . ', but we only support: '
-            . implode(',', $this->supportedAuthMethods)
-        );
+            sprintf('No supported authentication method found. The server supports these methods: %s, but we only support: %s',
+                    implode(', ', $this->_capability['sasl']),
+                    implode(', ', $this->supportedAuthMethods)));
     }