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

Address top 404s by adding redirect rules #635

Closed
9 tasks done
Tracked by #769
chalin opened this issue Jul 14, 2021 · 6 comments · Fixed by #868
Closed
9 tasks done
Tracked by #769

Address top 404s by adding redirect rules #635

chalin opened this issue Jul 14, 2021 · 6 comments · Fixed by #868
Assignees
Labels
bug Something isn't working e0-minutes Effort: < 60 min e1-hours Effort: < 8 hrs p2-medium

Comments

@chalin
Copy link
Contributor

chalin commented Jul 14, 2021

There have been about 400 page not found errors in the past 30 days (which isn't too bad). For example, the link to https://opentelemetry.io/releases/ mentioned in open-telemetry/opentelemetry-java#545 no longer exists. AFAICT there are no redirect rules currently.

Top 10 404s as of 2021-08-10:

@chalin chalin added the bug Something isn't working label Jul 14, 2021
@chalin
Copy link
Contributor Author

chalin commented Aug 10, 2021

Top 10 404s as of 2021-08-10 (see opening comment for text versions of the paths):

@chalin
Copy link
Contributor Author

chalin commented Aug 10, 2021

For example, @alolita's Nurturing Observability with Open Source post links to /about.

@chalin
Copy link
Contributor Author

chalin commented Aug 12, 2021

Ah, I'm seeing that the site uses Hugo aliases. IMHO, it is better to use Netlify redirects, even the Hugo doc's does so, see https://github.com/gohugoio/hugoDocs/blob/7cc1da82e757ba2c6d4d7df3ccd01d7d1adf132a/config.toml#L17-L18

@chalin
Copy link
Contributor Author

chalin commented Oct 20, 2021

@austinlparker - what did /releases/ used to refer to? What could we usefully redirect it to, if at all?

@austinlparker
Copy link
Member

I believe that went to what is now /status/.

@chalin
Copy link
Contributor Author

chalin commented Oct 28, 2021

@anuraaga wrote in open-telemetry/opentelemetry-java-examples#19:

I think for the quick fix we can redirect

/docs/java/getting_started/ -> https://opentelemetry.io/docs/java/manual_instrumentation/

/docs/java/automatic_instrumentation/ -> https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/README.md

Thanks for the feedback!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working e0-minutes Effort: < 60 min e1-hours Effort: < 8 hrs p2-medium
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants