Skip to content

Update geoip_update.php#1289

Closed
slsguerra wants to merge 1 commit intomailwatch:1.2from
slsguerra:patch-2
Closed

Update geoip_update.php#1289
slsguerra wants to merge 1 commit intomailwatch:1.2from
slsguerra:patch-2

Conversation

@slsguerra
Copy link
Copy Markdown
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
@ghost
Copy link
Copy Markdown

ghost commented Oct 14, 2023

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

@slsguerra
Copy link
Copy Markdown
Contributor Author

slsguerra 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
Copy Markdown
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.

2 participants