Skip to content

Improve redirected and broken links - #18

Merged
davetron5000 merged 2 commits into
davetron5000:mainfrom
noelleleigh:fix-links
Jun 1, 2025
Merged

davetron5000 merged 2 commits into
davetron5000:mainfrom
noelleleigh:fix-links

Conversation

@noelleleigh

@noelleleigh noelleleigh commented May 31, 2025

Copy link
Copy Markdown
Contributor
  • changelog.html: Fixed invalid URL scheme in link to book.
  • index.html: Replace W3.org link with the equivalent official WHATWG standard page.
  • index.html: Added trailing slash to avoid a redirect in book link.

- `changelog.html`: Fixed invalid URL scheme in link to book.
- `index.html`: Replace W3.org link with the equivalent official WHATWG
  standard page.
- `index.html`: Added trailing slash to avoid a redirect in book link.
@davetron5000

Copy link
Copy Markdown
Owner

Thanks! I'm curious about the trailing slashes. I don't get a redirect when omitting them in my browser, nor in curl:

> curl -v https://devbox.computer > /dev/null
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0* Host devbox.computer:443 was resolved.
* IPv6: (none)
* IPv4: 185.199.111.153, 185.199.110.153, 185.199.108.153, 185.199.109.153
*   Trying 185.199.111.153:443...
* Connected to devbox.computer (185.199.111.153) port 443
* ALPN: curl offers h2,http/1.1
* (304) (OUT), TLS handshake, Client hello (1):
} [320 bytes data]
*  CAfile: /etc/ssl/cert.pem
*  CApath: none
* (304) (IN), TLS handshake, Server hello (2):
{ [122 bytes data]
* (304) (IN), TLS handshake, Unknown (8):
{ [19 bytes data]
* (304) (IN), TLS handshake, Certificate (11):
{ [2625 bytes data]
* (304) (IN), TLS handshake, CERT verify (15):
{ [264 bytes data]
* (304) (IN), TLS handshake, Finished (20):
{ [36 bytes data]
* (304) (OUT), TLS handshake, Finished (20):
} [36 bytes data]
* SSL connection using TLSv1.3 / AEAD-CHACHA20-POLY1305-SHA256 / [blank] / UNDEF
* ALPN: server accepted h2
* Server certificate:
*  subject: CN=devbox.computer
*  start date: Apr 27 10:50:39 2025 GMT
*  expire date: Jul 26 10:50:38 2025 GMT
*  subjectAltName: host "devbox.computer" matched cert's "devbox.computer"
*  issuer: C=US; O=Let's Encrypt; CN=R11
*  SSL certificate verify ok.
* using HTTP/2
* [HTTP/2] [1] OPENED stream for https://devbox.computer/
* [HTTP/2] [1] [:method: GET]
* [HTTP/2] [1] [:scheme: https]
* [HTTP/2] [1] [:authority: devbox.computer]
* [HTTP/2] [1] [:path: /]
* [HTTP/2] [1] [user-agent: curl/8.7.1]
* [HTTP/2] [1] [accept: */*]
> GET / HTTP/2
> Host: devbox.computer
> User-Agent: curl/8.7.1
> Accept: */*
> 
* Request completely sent off
< HTTP/2 200 
< server: GitHub.com
< content-type: text/html; charset=utf-8
< last-modified: Wed, 10 Apr 2024 17:44:45 GMT
< access-control-allow-origin: *
< etag: "6616d00d-4dc8"
< expires: Sat, 31 May 2025 11:45:42 GMT
< cache-control: max-age=600
< x-proxy-cache: MISS
< x-github-request-id: DB88:793B7:16F7447:1881071:683AE98E
< accept-ranges: bytes
< age: 0
< date: Sat, 31 May 2025 11:35:42 GMT
< via: 1.1 varnish
< x-served-by: cache-iad-kiad7000146-IAD
< x-cache: MISS
< x-cache-hits: 0
< x-timer: S1748691343.839150,VS0,VE12
< vary: Accept-Encoding
< x-fastly-request-id: 32226d5fc3844a590bfa1b7bccc03c7b3b1a4c0b
< content-length: 19912
< 

@noelleleigh

Copy link
Copy Markdown
Contributor Author

Thanks! I'm curious about the trailing slashes. I don't get a redirect when omitting them in my browser, nor in curl:

Ah, you're right. My browser was doing it, but I should've just stuck with what LinkChecker told me.

@davetron5000
davetron5000 merged commit 967a213 into davetron5000:main Jun 1, 2025
@davetron5000

Copy link
Copy Markdown
Owner

This is now live - thanks again for the fix!

@noelleleigh
noelleleigh deleted the fix-links branch June 1, 2025 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants