Skip to content

FIX: rescue mmdb download failure#27556

Merged
tgxworld merged 2 commits intodiscourse:mainfrom
pfaffman:patch-9
Jun 27, 2024
Merged

FIX: rescue mmdb download failure#27556
tgxworld merged 2 commits intodiscourse:mainfrom
pfaffman:patch-9

Conversation

@pfaffman
Copy link
Copy Markdown
Contributor

If mmdb database fails to download a bootstrap fails. This is a trivial fix for that problem. A more elegant solution might check whether the dataabase was downloaded and provide a helpful error message.

If mmdb database fails to download a bootstrap fails. This is a trivial fix for that problem. A more elegant solution might check whether the dataabase was downloaded and provide a helpful error message.
@discoursebot
Copy link
Copy Markdown

This pull request has been mentioned on Discourse Meta. There might be relevant details there:

https://meta.discourse.org/t/rebuilding-always-fails-when-the-maxmind-daily-limit-is-exhausted-i-think-it-would-be-good-to-fix-this/312775/4

@SamSaffron
Copy link
Copy Markdown
Member

oh not linting, I am borderline ok with this @tgxworld ?

@jjaffeux jjaffeux changed the title rescue mmdb download failure FIX: rescue mmdb download failure Jun 27, 2024
@jjaffeux jjaffeux requested a review from tgxworld June 27, 2024 08:53
@jjaffeux
Copy link
Copy Markdown
Contributor

I fixed the linting, and requested a review from tgx.

@tgxworld tgxworld merged commit 6a5ee06 into discourse:main Jun 27, 2024
@pfaffman
Copy link
Copy Markdown
Contributor Author

Thanks, @jjaffeux and @tgxworld !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants