From:  alec@alec.pl
Date:  22 Jan 2015 19:33:21 Hong Kong Time
Newsgroup:  news.php.net/php.pear.bugs
Subject:  

[PEAR-BUG] Req #20491 [Fbk]: Redundant CAPABILITY requests

NNTP-Posting-Host:  null

Edit report at https://pear.php.net/bugs/bug.php?id=20491&edit=1

 ID:               20491
 Updated by:       alec@alec.pl
 Reported By:      alec at alec dot pl
 Summary:          Redundant CAPABILITY requests
 Status:           Feedback
 Type:             Feature/Change Request
 Package:          Net_Sieve
 Package Version:  1.3.4
 PHP Version:      Irrelevant
 Roadmap Versions: 
 New Comment:

So, there's no problem. The point is to not ask for them if they are
provided in a response. I will work on this and provide a patch after
the weekend.


Previous Comments:
------------------------------------------------------------------------

[2015-01-22 11:13:51] yunosh

No, it didn't advertise new capabilities after STARTTLS, so we had to
explicitly ask for those.

------------------------------------------------------------------------

[2015-01-21 13:32:12] alec

In which way it was broken? It returned wrong capabilities or what?

------------------------------------------------------------------------

[2015-01-21 12:59:50] yunosh

-Status: Open
+Status: Feedback
Both starting TLS as well es logging in can result in a changed
capabilities set. At least after STARTTLS the server must automatically
advertise the latest capabilities, but in some versions of Cyrus this
was broken, so we explicitely request those.

------------------------------------------------------------------------

[2015-01-21 12:50:42] alec

Description:
------------
In connection/login process we have at least two redundant CAPABILITY
calls. Here's how it looks now:

S: "IMPLEMENTATION" "Cyrus timsieved
git2.5+0-Kolab-2.5-96.el6.kolab_3.4"
S: "SASL" "PLAIN LOGIN"
S: "SIEVE" "comparator-i;ascii-numeric fileinto reject vacation
imapflags ...
S: "STARTTLS"
S: "UNAUTHENTICATE"
S: OK
C: CAPABILITY
S: "IMPLEMENTATION" "Cyrus timsieved
git2.5+0-Kolab-2.5-96.el6.kolab_3.4"
S: "SASL" "PLAIN LOGIN"
S: "SIEVE" "comparator-i;ascii-numeric fileinto reject vacation
imapflags ...
S: "STARTTLS"
S: "UNAUTHENTICATE"
S: OK
C: STARTTLS
S: OK "Begin TLS negotiation now"
STARTTLS negotiation successful
S: "IMPLEMENTATION" "Cyrus timsieved
git2.5+0-Kolab-2.5-96.el6.kolab_3.4"
S: "SASL" "PLAIN LOGIN"
S: "SIEVE" "comparator-i;ascii-numeric fileinto reject vacation
imapflags ...
S: "UNAUTHENTICATE"
S: OK
C: CAPABILITY
S: "IMPLEMENTATION" "Cyrus timsieved
git2.5+0-Kolab-2.5-96.el6.kolab_3.4"
S: "SASL" "PLAIN LOGIN"
S: "SIEVE" "comparator-i;ascii-numeric fileinto reject vacation
imapflags ...
S: "UNAUTHENTICATE"
S: OK
C: AUTHENTICATE "PLAIN" "AG1hcmsuZ2VybWFuQGV4YW1wbGUub3JnADEyMzQ1"
S: OK
C: CAPABILITY
S: "IMPLEMENTATION" "Cyrus timsieved
git2.5+0-Kolab-2.5-96.el6.kolab_3.4"
S: "SIEVE" "comparator-i;ascii-numeric fileinto reject vacation
imapflags ...
S: "UNAUTHENTICATE"
S: OK

------------------------------------------------------------------------


-- 
Edit this bug report at https://pear.php.net/bugs/bug.php?id=20491&edit=1