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	>>>> Implemented. >>>> >>>> Anyway you can test the current Git code in your setup? >>> >>> That was fast! >>> Of course I will test, but it may take some days, I'm not sure I can >>> squeeze it in today. I'll give you feedback till end of week at >>> latest. >>> Torben >> >> Two question rise while reading the code: >> - What is the intent of Audodiscover.php lines 130:135 ? >> - How are the params of the outlook autodiscover set? Only via the >> hook or also via the actviesync configuration tab ? Just curious... >> >> Torben >> > > Now I got the reason why it is not working: > > around line 130, you test the responseschema to be empty. > > This is not correct: for every autodiscover query the accetable > response schema is delivered. Thus this variable should always be > set, otherwise our parsing of the request is wrong. > > Indeed, as I tested with outlook 2010 and also with the Microsoft > "Remote Connectivity Analyzer" ( Connectivity Tests for Microsoft > Office Outlook ), I discovered that with the RCA, the response schema > is empty, because the string has different positions in Outlook and > in RCA. This was new for me and was not adressed yet. > > In my opinion we have to apply two changes to Autodiscover.php to get > it working: > 1. refactor line 130:135 to avoid > "!empty($properties['response_schema'])" - which should never be > emtpy.. As I said, I have no clue about the reason of this if-clause > at lines 130:135 > 2. Make the repsonse schema more robust. It is not sufficient to > extract values at certain place, but we have to parse for > ACCEPTABLERESPONSESCHEMA to get the right value. > > Please find below the request data send by RCA and by Outlook to get > an idea how to fetch the response schema. this depub output is done > by print:r() and thus represents the array structure > > ------------------------ > values send by RCA: > [0] => Array > ( > [tag] => AUTODISCOVER > [type] => open > [level] => 1 > [attributes] => Array > ( > [XMLNS:XSD] => http://www.w3.org/2001/XMLSchema > [XMLNS:XSI] => http://www.w3.org/2001/XMLSchema-instance > [XMLNS] => > http://schemas.microsoft.com/exchange/autodiscover/outlook/requestschema/2006 > ) > > [value] => > > ) > > [1] => Array > ( > [tag] => REQUEST > [type] => open > [level] => 2 > [value] => > > ) > > [2] => Array > ( > [tag] => EMAILADDRESS > [type] => complete > [level] => 3 > [value] => test2013@dannhauer.de > ) > > [3] => Array > ( > [tag] => REQUEST > [value] => > > [type] => cdata > [level] => 2 > ) > > [4] => Array > ( > [tag] => ACCEPTABLERESPONSESCHEMA > [type] => complete > [level] => 3 > [value] => > http://schemas.microsoft.com/exchange/autodiscover/outlook/responseschema/2006a > ) > > [5] => Array > ( > [tag] => REQUEST > [value] => > > [type] => cdata > [level] => 2 > ) > > [6] => Array > ( > [tag] => REQUEST > [type] => close > [level] => 2 > ) > > [7] => Array > ( > [tag] => AUTODISCOVER > [value] => > > [type] => cdata > [level] => 1 > ) > > [8] => Array > ( > [tag] => AUTODISCOVER > [type] => close > [level] => 1 > ) > --------------------- > values send by Outlook: > [0] => Array > ( > [tag] => AUTODISCOVER > [type] => open > [level] => 1 > [attributes] => Array > ( > [XMLNS] => > http://schemas.microsoft.com/exchange/autodiscover/outlook/requestschema/2006 > ) > > ) > > [1] => Array > ( > [tag] => REQUEST > [type] => open > [level] => 2 > ) > > [2] => Array > ( > [tag] => EMAILADDRESS > [type] => complete > [level] => 3 > [value] => test2013@dannhauer.de > ) > > [3] => Array > ( > [tag] => ACCEPTABLERESPONSESCHEMA > [type] => complete > [level] => 3 > [value] => > http://schemas.microsoft.com/exchange/autodiscover/outlook/responseschema/2006a > ) > > [4] => Array > ( > [tag] => REQUEST > [type] => close > [level] => 2 > ) > > [5] => Array > ( > [tag] => AUTODISCOVER > [type] => close > [level] => 1 > ) > > Thanks Torben
Attachment
Watch this ticket

Saved Queries