Tickets :: Comment on [#11639] ActiveSync :: Extend the Autodiscover feature to return also valid answers to MS Outlook queries

* Your Email Address
* Spam protection	Enter the letters below: .__ .__. __.. .___ [__)[__](__ \| [__ [__)\| \|.__)\|___[___
Comment	>> How about now? >> >> >> http://lists.horde.org/archives/commits/2012-November/017210.html >> >> http://lists.horde.org/archives/commits/2012-November/017212.html > > Hi, > The respond schema is now detected correctly for both, Outlook and RCA. > > > Regarding teh logic fix: Maybe I'm missing the point, but for me it > does make still no sense. > > My observation was, that outlook 2013 performs two independet > autodiscover steps: It tries to discover a mobilesync setting, and it > tries to discover a conventional Outlook setting as Outlook 2010 and > Outlook 2007 do. > > Some suggestions for Autodiscover.php: > - Line 113 and 136 check both for empty repsonse schema. this could > be done directly after entering the _buildResponseString(..) > function to make the rest of the function easier to read. > - Line 135 opens a conditionalblock for the "autodiscover/outlook" > schema, so within this block, it does not make sense to check for a > mobilesync schema as done in lines 140:144. -> what was the intention > of the line? In my opinion we must drop that lines, because the > comparison in line 140 is always true and thus the function returns a > failure response. > > Bye the way: > the failure response has always the response schema for a > mobilesync, so RCA seems not to unterstand that an error occured. the > _buildFailureResponse(..) function should use the correct response > schema in line 216 to allow Outlook & RCA to unterstand that the > error code is the response to teir query. > > Torben
Attachment
Watch this ticket

Saved Queries