| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.3.8 |
| Bug #4243 | Problems assessing capabilities. | ||
|---|---|---|---|
| Submitted: | 2005-04-28 21:06 UTC | ||
| From: | rlpowell at digitalkingdom dot org | Assigned: | doconnor |
| Status: | Closed | Package: | Net_POP3 |
| PHP Version: | 4.3.10 | OS: | Debian Linux |
| Roadmaps: | (Not assigned) | ||
[2005-04-28 21:06 UTC] rlpowell at digitalkingdom dot org
Description:
------------
The following is what I had to do to make net_pop3.php not crash when we inserted into TikiWiki:
Index: ./lib/webmail/net_pop3.php
===================================================================
RCS file: /cvsroot/tikiwiki/tiki/lib/webmail/Attic/net_pop3.php,v
retrieving revision 1.1.2.1
retrieving revision 1.1.2.3
diff -r1.1.2.1 -r1.1.2.3
36c36
< // $Id: net_pop3.php,v 1.1.2.1 2005/04/23 20:47:04 mose Exp $
---
> // $Id: net_pop3.php,v 1.1.2.3 2005/04/28 20:56:20 rlpowell Exp $
295c295
< $this->_capability['implementation'] = $matches[3];
---
> $this->_capability['implementation'] = $matches[1];
298c298
< $this->_capability['sasl'] = preg_split('/\s+/', $matches[3]);
---
> $this->_capability['sasl'] = preg_split('/\s+/', $matches[1]);
301c301
< $this->_capability[$capa] = $matches[2];
---
> $this->_capability[$capa] = $matches[1];
337a338,339
> $serverMethods = array();
>
339c341
< $serverMethods=$this->_capability['sasl'];
---
> $serverMethods[] = $this->_capability['sasl'][0];
Comments
[2005-05-20 09:45 UTC] pearbug20050520 at pech dot cz
This is the problem with undefined index, which has no direct connection to this bug, but should be solved with this bug. You can for example try it against dovecot.
Thanks.
Jirka Pech
--- POP3.php.orig 2005-05-20 11:42:46.317135367 +0200
+++ POP3.php 2005-05-20 11:39:51.590440164 +0200
@@ -295,7 +295,9 @@
$this->_capability['implementation'] = $matches[3];
break;
case 'sasl':
+ if (isset($matches[3])) {
$this->_capability['sasl'] = preg_split('/\s+/', $matches[3]);
+ }
break;
default :
$this->_capability[$capa] = $matches[2];
[2005-05-20 18:20 UTC] rlpowell at digitalkingdom dot org
Just for the record, dovecot is what I was using as well.
Also, I suggest adding to the if in the second patch below the following:
} else {
$this->_capability['sasl'] = $matches[1];
}
[2007-03-27 12:04 UTC] cweiske
(Christian Weiske)
Could you please enable debug with $pop3->setDebug(true); and paste the echoed debug messages so that I can try to see what is really wrong/different with dovecot responses?
[2007-09-22 20:52 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2011-12-09 15:45 UTC] doconnor
(Daniel O'Connor)
-Status: Open
+Status: Closed
-Assigned To:
+Assigned To: doconnor
This bug has been fixed in SVN.
If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET).
If this was a problem with the pear.php.net website, the change should be live shortly.
Otherwise, the fix will appear in the package's next release.
Thank you for the report and for helping us make PEAR better.
Applied the attached patch