Skip to content

Fix CI/CD link

William Chia requested to merge williamchia-master-patch-72747 into master

@cblake Since you mentioned this as a learning op in slack, I thought I'd try to provide some comments:

The bug here was a subtle one - a missing / at the front of the link path.

solutions/dev-sec-ops/ vs /solutions/dev-sec-ops/

The difference in behavior between these is due to URLs using the same mechanics as UNIX directory traversal.

Nutshell

  • no slash at the front means "tack this on to the URL of the current page".
    • e.g. solutions/dev-sec-ops/ translates to https://about.gitlab.com/2019/04/12/gitlab-is-an-sca-contender/solutions/dev-sec-ops
  • a slash at the front mean "start at the root directory for the website".
  • In our case that's https://about.gitlab.com/.
  • e.g. /solutions/dev-sec-ops/ translates to https://about.gitlab.com/solutions/dev-sec-ops/
  • More fun info here on relative links: http://www.dirigodev.com/blog/seo-web-best-practices/relative-vs-absolute-urls-seo/

Merge request reports