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

Update geoip_update.php #1289

Closed
wants to merge 1 commit into from
Closed

Conversation

HierraStrunger
Copy link
Contributor

Permalinks from maxmind website says:
/app/geoip_download?edition_id=GeoLite2-Country&license_key=YOUR_LICENSE_KEY&suffix=tar.gz

Permalinks from maxmind website says:
/app/geoip_download?edition_id=GeoLite2-Country&license_key=YOUR_LICENSE_KEY&suffix=tar.gz
@shawniverson
Copy link
Member

@HierraStrunger Are you seeing an issue with geoip_update.php? The order of the URL parameters should not matter.

@HierraStrunger
Copy link
Contributor Author

HierraStrunger commented Oct 14, 2023

Yes, the message error is:

Error occurred while downloading GeoIP data file: Response could not be parsed

And the file is not downloaded. After change geoip_update.php, it works

@endelwar endelwar closed this in f8b2962 Oct 16, 2023
@endelwar
Copy link
Member

I've fixed in Request library the regex that parses HTTP/2 header, which presents a trailing space not followed by text like in HTTP 1.x

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

Successfully merging this pull request may close these issues.

3 participants