Skip to content

Commit

Permalink
Enforce sending of ID request after login.
Browse files Browse the repository at this point in the history
This request was queued instead, and sent during shutdown scripts. For some
reason the streams were empty during shutdown here. Mabye streams are closed or
emptied during shutdown and for some reason this happened before the other
shutdown scripts here. I don't know. This fixes it.
  • Loading branch information
yunosh committed Dec 8, 2015
1 parent 03e47a7 commit bad7f2e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
2 changes: 2 additions & 0 deletions framework/Imap_Client/lib/Horde/Imap/Client/Base.php
Expand Up @@ -832,6 +832,8 @@ public function login()
if ($this->getParam('id')) {
try {
$this->sendID();
/* ID is queued - force sending the queued command. */
$this->_sendCmd($this->_pipeline());
} catch (Horde_Imap_Client_Exception_NoSupportExtension $e) {
// Ignore if server doesn't support ID extension.
}
Expand Down
4 changes: 3 additions & 1 deletion framework/Imap_Client/package.xml
Expand Up @@ -21,7 +21,8 @@
</stability>
<license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
<notes>
*
* [jan] Fix broken ID requests under certain circumstances.
* [jan] Fix parsing IMAP responses with tilde characters.
</notes>
<contents>
<dir baseinstalldir="/" name="/">
Expand Down Expand Up @@ -2958,6 +2959,7 @@
<date>2015-11-22</date>
<license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
<notes>
* [jan] Fix broken ID requests under certain circumstances.
* [jan] Fix parsing IMAP responses with tilde characters.
</notes>
</release>
Expand Down

0 comments on commit bad7f2e

Please sign in to comment.