PHP Parse error: syntax error, unexpected '->' (T_OBJECT_OPERATOR) in vendor\pragmarx\firewall\src\Firewall.php on line 584

[jmichaelterenin]

Getting this error when attempting to perform: php artisan firewall:tables (no problems) php artisan migrate (error occurs)

I assumed you were referencing the geoip2/geoip2 package, even though you state that it's optional in the documentation, so I required it in my composer, performed a composer update, dump-autoload, and returning to the migration step, I still get the error.

Can you assist? Thanks

[antonioribeiro]

Which version are you using?

[jmichaelterenin]

Hi Antonio, Sorry for not including: Laravel 5.4, "pragmarx/firewall" : "^2.0", "geoip2/geoip2" : "~2.0",

[antonioribeiro]

Line 584 related to the updategeoip command

And it is not executed elsewhere.

[jmichaelterenin]

Sure, I saw that, but I'm sorry, I'm unfamiliar with the Geoip package, not sure what that means, I don't even plan on using any country blacklisting, just ip address ...

[antonioribeiro]

So, the problem was during "artisan migrate" or "artisan firewall:updategeoip"?

Could you, please send us the whole error message, and the call stack? Should appear in your laravel.log

[jmichaelterenin]

`[2017-08-25 12:13:56] local.ERROR: exception 'Symfony\Component\Debug\Exception\FatalErrorException' with message 'syntax error, unexpected '->' (T_OBJECT_OPERATOR)' in C:\xampp\htdocs\explore\vendor\pragmarx\firewall\src\Firewall.php:584 Stack trace:

0 {main} `

That is the entire call stack. occurs with "artisan migrate"

[antonioribeiro]

You get nothing in storage/logs/laravel.log?

[jmichaelterenin]

Hi Antonio, What is highlighted in code (with the time stamp), is what I get in laravel.log

[antonioribeiro]

So files must have not downloaded as they should. Try to remove vendor/pragmarx and then

composer update

again

[jmichaelterenin]

ok, I'll let you know if it gets resolved with a fresh download, thanks

[justplayingames]

I see the same error

php version 5.6.24 pragmarx/firewall version": "v1.1.0",

It acts a syntax error. I get the error when attempting to serve my index page using php artisan serve. This route is using the firewall package.

A quick fix was to change line 521 to do it in two lines:

$updater = new GeoIpUpdater;
$success = $updater->updateGeoIpFiles($this->config->get('geoip_database_path'));

I confirmed the same fix on line 584 of pragmarx/firewall version 2.0.1

[antonioribeiro]

Just tagged version 2.0.2.

Thank you all.

[jmichaelterenin]

@justplayingames Thank you for assisting @antonioribeiro AOK now, thanks!