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

Point to wiki page about certs on cert errors #2279

Merged
merged 1 commit into from Feb 13, 2018

Conversation

HebaruSan
Copy link
Member

Problem

As #2278 points out, the error message for certificate errors is outdated:

Oh no! Our download failed with a certificate error!

If you're on Linux, try running:
	mozroots --import --ask-remove
on the command-line to update your certificate store, and try again.

According to our own wiki pages, mozroots has been superseded by cert-sync since Mono 3.12.0 in early 2015, but the error message still suggests mozroots.

Changes

This pull request updates it to recommend the wiki page about certs instead:

This page was created just now based on the information previously scattered across each Linux page about certs, which now link to the new page. This way instead of updating the code to keep up with changes in Mono, we can just update the wiki, and everyone has the latest info.

Since we can check the user's platform directly, "If you're on Linux" is removed.

@HebaruSan HebaruSan added Pull request Linux Issues specific for Linux Mono Issues specific for Mono Documentation Issues affecting documentation labels Feb 11, 2018
@HebaruSan HebaruSan mentioned this pull request Feb 11, 2018
@politas politas merged commit b83f758 into KSP-CKAN:master Feb 13, 2018
@HebaruSan HebaruSan deleted the fix/cert-error-msg branch February 13, 2018 02:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Issues affecting documentation Linux Issues specific for Linux Mono Issues specific for Mono Pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants