Skip to content
This repository has been archived by the owner on Oct 18, 2020. It is now read-only.

Nicer error message when no certs could be found for a site #88

Open
saxbophone opened this issue Apr 16, 2016 · 3 comments
Open

Nicer error message when no certs could be found for a site #88

saxbophone opened this issue Apr 16, 2016 · 3 comments

Comments

@saxbophone
Copy link

I tried this with my own very simple site, which has no HTTPS at all.
https://shaaaaaaaaaaaaa.com/check/saxbophone.com

Instead of saying no certs could be found or something similar, it says Argh, there has been an error and advises me to check the console.

Checking the console reveals that no certs could be found, but it'd be nice to see this in the UI.

@konklone
Copy link
Owner

It's a good idea! If anyone wants to tackle this, they're welcome to - I won't have time to do this in the near future.

@rajatRS
Copy link

rajatRS commented Oct 2, 2020

I have made a pull request, fixing this. Please, see If its good enough for you to merge that and do let me know your views. :)

@saxbophone
Copy link
Author

I should hasten to add, that saxbophone.com may no longer present a good test case as I now have HTTPS on that domain...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants