Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PHP 8.1 Compatibility? #13

Open
justbaron opened this issue Feb 28, 2022 · 3 comments
Open

PHP 8.1 Compatibility? #13

justbaron opened this issue Feb 28, 2022 · 3 comments

Comments

@justbaron
Copy link

PHP version: PHP 8.1.1 (cli) (built: Dec 15 2021 10:31:43) (ZTS Visual C++ 2019 x64)
Microsoft Exchange version: VERSION_2013_SP1

Description of problem:
Is there a compatible version for PHP 8.1?
Using the jamesiarmes/php-ews classes and attempting to "SendMail", my Apache instance crashes.
I have tried adding the #[\ReturnTypeWillChange] attribute above both "__doRequest" and "__getLastRequestHeaders" functions, which suppress the errors, but still crashes my Apache instance.

Error.log:

PHP Deprecated:  Return type of jamesiarmes\PhpNtlm\SoapClient::__doRequest($request, $location, $action, $version, $one_way = 0) should either be compatible with SoapClient::__doRequest(string $request, string $location, string $action, int $version, bool $oneWay = false): ?string, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in ..\composer\vendor\jamesiarmes\php-ntlm\src\SoapClient.php on line 71

PHP Deprecated:  Return type of jamesiarmes\PhpNtlm\SoapClient::__getLastRequestHeaders() should either be compatible with SoapClient::__getLastRequestHeaders(): ?string, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in ..\composer\vendor\jamesiarmes\php-ntlm\src\SoapClient.php on line 106
@satalaondrej
Copy link
Contributor

@justbaron Check this #14 out, it works this way on php 8.1 for me

@jsakars
Copy link

jsakars commented May 11, 2022

@jamesiarmes hi! any chance to merge this?

@giox069
Copy link

giox069 commented Aug 30, 2022

Same problem here after upgrading to PHP8.1.
What is preventing from #14 to be merged ? (I did not test it)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants