[PR #1857] [MERGED] Make homepage link simpler for end users #2657

New issue

Closed

opened 2026-03-16 11:01:07 +03:00 by kerem · 0 comments

kerem commented

2026-03-16 11:01:07 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hickory-dns/hickory-dns/pull/1857
Author: @HLFH
Created: 12/15/2022
Status: ✅ Merged
Merged: 12/20/2022
Merged by: @bluejekyll

Base: main ← Head: fix-homepage-link

📝 Commits (1)

1baf5d5 make homepage link simpler for end users

📊 Changes

10 files changed (+10 additions, -10 deletions)

View changed files

📝 bin/Cargo.toml (+1 -1)
📝 crates/async-std-resolver/Cargo.toml (+1 -1)
📝 crates/client/Cargo.toml (+1 -1)
📝 crates/proto/Cargo.toml (+1 -1)
📝 crates/recursor/Cargo.toml (+1 -1)
📝 crates/resolver/Cargo.toml (+1 -1)
📝 crates/server/Cargo.toml (+1 -1)
📝 tests/compatibility-tests/Cargo.toml (+1 -1)
📝 tests/integration-tests/Cargo.toml (+1 -1)
📝 util/Cargo.toml (+1 -1)

📄 Description

I advise to remove the deprecated www. subdomain from the homepage link.

I quote a Google staffer:

"www" is now considered a "trivial" subdomain.

The web server Nginx used by trust-dns.org is correctly configured to redirect to https://trust-dns.org/.

curl http://trust-dns.org 
<html>
<head><title>301 Moved Permanently</title></head>
<body>
<center><h1>301 Moved Permanently</h1></center>
<hr><center>nginx</center>
</body>
</html>

curl https://www.trust-dns.org/
<html>
<head><title>301 Moved Permanently</title></head>
<body>
<center><h1>301 Moved Permanently</h1></center>
<hr><center>nginx</center>
</body>
</html>

While the Nginx configuration of the website should still support it, the deprecated convention should no longer be advertised in the files.

Secondly, index.html should also be removed from the commits as advised here.

Nginx web server / website issue separate from this PR:

As seen here, what needs to be also done for the trust-dns.org conf. of the Nginx web server should be something like that:

location / {
        #remove index.html from request
        rewrite ^/(.*)/index(\.html)?$ /$1 permanent;    
        rewrite ^/(.*)index(\.html)?$ /$1 permanent;        

        #remove .html from request
        rewrite ^/(.*)\.html$ /$1 permanent;
        
        try_files $uri ${uri}/index.html ${uri}.html =404;
    }

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/hickory-dns/hickory-dns/pull/1857 **Author:** [@HLFH](https://github.com/HLFH) **Created:** 12/15/2022 **Status:** ✅ Merged **Merged:** 12/20/2022 **Merged by:** [@bluejekyll](https://github.com/bluejekyll) **Base:** `main` ← **Head:** `fix-homepage-link` --- ### 📝 Commits (1) - [`1baf5d5`](https://github.com/hickory-dns/hickory-dns/commit/1baf5d56e436650b6b682ec152c7ecb237bea34e) make homepage link simpler for end users ### 📊 Changes **10 files changed** (+10 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `bin/Cargo.toml` (+1 -1) 📝 `crates/async-std-resolver/Cargo.toml` (+1 -1) 📝 `crates/client/Cargo.toml` (+1 -1) 📝 `crates/proto/Cargo.toml` (+1 -1) 📝 `crates/recursor/Cargo.toml` (+1 -1) 📝 `crates/resolver/Cargo.toml` (+1 -1) 📝 `crates/server/Cargo.toml` (+1 -1) 📝 `tests/compatibility-tests/Cargo.toml` (+1 -1) 📝 `tests/integration-tests/Cargo.toml` (+1 -1) 📝 `util/Cargo.toml` (+1 -1) </details> ### 📄 Description I advise to remove the deprecated www. subdomain from the homepage link. I [quote a Google staffer](https://bugs.chromium.org/p/chromium/issues/detail?id=881410#c1): > "www" is now considered a "trivial" subdomain. The web server Nginx used by trust-dns.org is correctly configured to redirect to https://trust-dns.org/. ``` curl http://trust-dns.org <html> <head><title>301 Moved Permanently</title></head> <body> <center><h1>301 Moved Permanently</h1></center> <hr><center>nginx</center> </body> </html> curl https://www.trust-dns.org/ <html> <head><title>301 Moved Permanently</title></head> <body> <center><h1>301 Moved Permanently</h1></center> <hr><center>nginx</center> </body> </html> ``` While the Nginx configuration of the website should still support it, the deprecated convention should no longer be advertised in the files. Secondly, index.html should also be removed from the commits [as advised here](https://webmasters.stackexchange.com/a/122313). --- #### Nginx web server / website issue separate from this PR: [As seen here](https://webmasters.stackexchange.com/a/140434), what needs to be also done for the trust-dns.org conf. of the Nginx web server should be something like that: ``` location / { #remove index.html from request rewrite ^/(.*)/index(\.html)?$ /$1 permanent; rewrite ^/(.*)index(\.html)?$ /$1 permanent; #remove .html from request rewrite ^/(.*)\.html$ /$1 permanent; try_files $uri ${uri}/index.html ${uri}.html =404; } ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>