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.

Sign up for GitHub

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

Jump to bottom

Documentation: Non-HTTPS Links Do not Work #1594

Closed
sanssecours opened this issue Aug 31, 2017 · 4 comments
Closed

Documentation: Non-HTTPS Links Do not Work #1594

sanssecours opened this issue Aug 31, 2017 · 4 comments
Assignees
@BernhardDenner
Labels
bug usability
Milestone
0.8.21

Comments

@sanssecours
Copy link
Member

Steps to Reproduce

Open http://doc.libelektra.org/api/current/html in your favorite browser

Expected Result

The web browser redirects to the page https://doc.libelektra.org/api/current/html

Actual Result

<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
<html><head>
<title>404 Not Found</title>
</head><body>
<h1>Not Found</h1>
<p>The requested URL /api/current/html was not found on this server.</p>
<hr>
<address>Apache/2.4.10 (Debian) Server at doc.libelektra.org Port 80</address>
</body></html>
@sanssecours sanssecours added this to the 0.8.20 milestone Aug 31, 2017
@markus2330
Copy link
Contributor

Thank you for reporting! I did not notice because my browser still show the page (maybe because of https anywhere or because the old redirect is still cached).

How did you end up at the broken link? (I replaced the ones in the repo, but maybe there are more.)

Btw. should we change to a different 404 page?

markus2330 pushed a commit that referenced this issue Sep 3, 2017
doc: replace "http://doc.libelektra.org" "https://doc.libelektra.org"
56fadea 
see #1594
@sanssecours
Copy link
Member Author

How did you end up at the broken link?

The main page of the API documentation contains two links to the current and latest Doxygen documentation. One of them was – and in the homepage version still is – https://doc.libelektra.org/api/current/html.

Btw. should we change to a different 404 page?

If you have a really good idea on how this page should look, then why not. However, I am totally fine with the current version. In my opinion 404 pages are always annoying, even if they are well designed.

@markus2330
Copy link
Contributor

and in the homepage version still is

Yes, the homepage cannot be rebuild at the moment. Next to fixing the redirecting so that external links work, we should also update our website to only use https links.

If you have a really good idea on how this page should look, then why not. However, I am totally fine with the current version. In my opinion 404 pages are always annoying, even if they are well designed.

I was more thinking about that it should contain some information that the site is related to Elektra and this issue tracker should be used to report the problem. If someone comes up with an design, we can use that, too. We already have a designed 404 page for errors within the homepage: https://www.libelektra.org/error/404

BernhardDenner pushed a commit to BernhardDenner/libelektra that referenced this issue Oct 14, 2017
@BernhardDenner
doc: replace "http://doc.libelektra.org" "https://doc.libelektra.org"
86d7e5e 
see ElektraInitiative#1594
@markus2330 markus2330 modified the milestones: 0.8.20, 0.8.21 Oct 31, 2017
@markus2330
Copy link
Contributor

This seems to work for me now. Please reopen if there are still problems.

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

@BernhardDenner BernhardDenner

Labels
bug usability
Projects
None yet
Milestone
0.8.21
Development

No branches or pull requests
3 participants
@sanssecours @BernhardDenner @markus2330