-
Notifications
You must be signed in to change notification settings - Fork 45
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
Investigate / resolve link errors #341
Comments
Hello @mjang |
Great! I've assigned the issue to you. I look forward to your contributions! |
Hello @mjang , I was able to look at the workflow file for the link checker and also the corresponding links throwing the errors. Proposed solution: Replacing the link on the workflow file with the correct one would help resolve this. |
@FaithKovi That would be great! When I've researched these types of issues, I've:
Example, here's an excerpt from the linked 1_nginx_plus.txt file:
The broken link is at the following Based on the context, I know I'm looking for a link to docs.nginx.com or f5.com that describes load balancers. With some "googling," I find https://www.f5.com/glossary/load-balancer Solution: propose a PR to replace the link in https://docs.nginx.com/nginx/admin-guide/load-balancer/tcp-udp-load-balancer/ with https://www.f5.com/glossary/load-balancer. |
Describe the bug
CI/CD job https://github.com/nginx/documentation/actions/runs/14190446169/job/39753719002 shows link errors:
404 Not Found: 22 instances
301 Moved Permanently: 13 instances
Collected in the following file:
1_nginx-plus.txt
To reproduce
Rerun CI/CD job
Expected behavior
Minimum: no 404s
Ideal: no 301s either
Note
Possible "Good first issue" if I can figure out a way to "explain" the process
The text was updated successfully, but these errors were encountered: