Return value of Tuupola\\Middleware\\CorsMiddleware::origin() must be an instance of Tuupola\\Middleware\\void, none returned

[tuupola]

Return value of Tuupola\\Middleware\\CorsMiddleware::origin() must be an instance of Tuupola\\Middleware\\void, none returned

[tuupola]

What is the PHP version? PHP 7.1 is the minimal requirement for the CORS middleware.

[controlnocontrol]

We’re running PHP 7.2 on Apache 2.4.

…

On Jun 13, 2018, at 17:44, Mika Tuupola ***@***.***> wrote: What is the PHP version? PHP 7.1 is the minimal requirement for the CORS middleware. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

[tuupola]

Ok that is weird. Unit tests pass with PHP 7.2 and I am also using the middleware myself with PHP 7.2.

Have the files in vendor/ been edited manually? The method should look like following:

https://github.com/tuupola/cors-middleware/blob/master/src/CorsMiddleware.php#L159-L165

[controlnocontrol]

I figured the issue out. My host's cPanel was trying to add some code to the root's .htaccess file to dictate the PHP version. There was some conflict but now all seems fine! The weird that was that the server was still running PHP7.2 when dumping phpinfo. BTW I'm running Craft 3 CMS and SLIM on the same environment and everything seems to be working okay!

And no I haven't edited the vendor files manually. That wouldn't be a very wise thing to do. :)