Missing RequestServerVersion SOAP header

[jamesiarmes]

What steps will reproduce the problem?

1. FindFolder request with DistinguishedFolderId on publicfoldersroot and Traversal on Shallow.

Expected output: succesful response

Got soap error: The request is valid but does not specify the correct server version in the RequestServerVersion SOAP header. Ensure that the RequestServerVersion SOAP header is set with the correct RequestServerVersionValue.

Quick fix: to NTMLSOAPClient_Exchange::__construct(), I added this:

<?php
$soapHeader = new SoapHeader(
    '<a href="http://schemas.microsoft.com/exchange/services/2006/types'" rel="nofollow">http://schemas.microsoft.com/exchange/services/2006/types'</a>,
    'RequestServerVersion Version=&quot;Exchange2007_SP1&quot;'
);
$this-&gt;__setSoapHeaders($soapHeader);

Obviously, the Exchange2007_SP1 part should be configurable.

Originally posted by kristof.coomans on Google Code: http://code.google.com/p/php-ews/issues/detail?id=3

[jamesiarmes]

I find it odd that I haven't run into this issue before. I will verify that this is all that is required and take care of it this weekend.

Originally posted by jamesiarmes on Google Code: http://code.google.com/p/php-ews/issues/detail?id=3#c1

[jamesiarmes]

Ah, this fix may work with the ExchangeImpersonation, however i'm unsure, how to impliment this on NTLMSoapClient_Exchange , any thoughts

Originally posted by goresci on Google Code: http://code.google.com/p/php-ews/issues/detail?id=3#c2

[jamesiarmes]

Hi, I've attached a patch that fixes this issue (we were affected as well.)

Originally posted by jon.kloske.uq on Google Code: http://code.google.com/p/php-ews/issues/detail?id=3#c3
File attached: php-ews_issue3_fix.diff

[jamesiarmes]

forgot to mention, to use the new code you need to specify the new optional exchange server version parameter to the ExchangewWebServices constructor, like so:

<?php
$ews = new ExchangeWebServices('server', 'username', 'password', EWSType_ExchangeVersionType::Exchange2007_SP1);

Originally posted by jon.kloske.uq on Google Code: http://code.google.com/p/php-ews/issues/detail?id=3#c4

[jamesiarmes]

This has been merged into the master branch. I made some changes to your patch, the biggest change being that I added the constants directly to the ExchangeWebServices class rather than adding a new class. I have also added you to the contributors file, thank you for your contribution.

Originally posted by jamesiarmes on Google Code: http://code.google.com/p/php-ews/issues/detail?id=3#c5