search

antonioribeiro / firewall

Firewall package for Laravel applications
BSD 3-Clause "New" or "Revised" License
1.39k stars 162 forks source link

Laravel 9 and PHP 8 patch #180

Closed liepumartins closed 2 years ago

liepumartins commented 2 years ago

Resolves #179

Deleted composer.lock to avoid errors and confusion. Should have been deleted long ago aed0dcd9df40148629b623f55f9ef7cdcd7332ab

Added new PHP versions to Travis CI

Removed Guzzle dependency, since nothing in the code uses guzzle.

liepumartins commented 2 years ago

Ah, Scrutinizer CI check fails with this. And Travis is not used at all?

1) PragmaRX\Firewall\Tests\ArtisanTest::testUpdateGeoip
ErrorException: gzopen(/home/scrutinizer/build/tests/geoipdb/GeoLite2-City.mmdb.gz): failed to open stream: No such file or directory

From what I can gather, it is because the URL of GeoIP database https://github.com/antonioribeiro/support/blob/df3714de795d06e6b68c66f9a8b80bab50f9b6ef/src/GeoIp/Updater.php#L7 does not work since 2019 and requires registration: https://dev.maxmind.com/geoip/geolite2-free-geolocation-data