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

Fixes broken links generated on relative URLs #215

Open
wants to merge 30 commits into
base: master
Choose a base branch
from

Conversation

mchurichi
Copy link
Member

@mchurichi mchurichi commented Apr 12, 2021

Fixes #205

Preview site: https://deploy-preview-215--spiffe.netlify.app/docs/latest/microservices/envoy-jwt/readme/
Broken before: https://spiffe.io/docs/latest/microservices/envoy-jwt/readme/ (broken link on SPIRE Envoy-X.509 Tutorial, on the first sentence)

Signed-off-by: Maximiliano Churichi [email protected]

@netlify
Copy link

netlify bot commented Apr 19, 2021

Deploy preview for spiffe ready!

Built with commit f10a384

https://deploy-preview-215--spiffe.netlify.app

Copy link
Member

@Andres-GC Andres-GC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍦

Maximiliano Churichi added 4 commits May 4, 2021 11:42
Signed-off-by: Maximiliano Churichi <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
@sanderson042
Copy link
Contributor

Testing this change, I'm no longer getting 404 errors from GitHub. But if the source file from GitHub has a link to another GitHub page without the file name, the GitHub version of the page is displayed. If the link includes the file name, the spiffe.io version of the GitHub file is displayed as expected.

For example, the spiffe/spire-tutorials/k8s/envoy-jwt/README.md file on GitHub has both variants.

In that file, this Markdown URL format without the target file name displays the GitHub version of the file:
[SPIRE Envoy-X.509 Tutorial](../envoy-x509/)

This Markdown URL format with the target file name displays the spiffe.io version of the file, which is the desired behavior:
[SPIRE Envoy-X.509 Tutorial](../envoy-x509/README.md)

Maximiliano Churichi and others added 15 commits May 17, 2021 09:56
Signed-off-by: Maximiliano Churichi <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
* In configuring.md, replaced link to k8s Token Review API with
  different location that doesn't include a version number

* In libraries.md, replaced link to archived c-spiffe library with new
  c-spiffe library under development by HPE

Signed-off-by: Steve Anderson <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
Maximiliano Churichi and others added 9 commits June 10, 2021 10:56
Signed-off-by: Maximiliano Churichi <[email protected]>
Thanks Max and Andrew for your tweaks to this doc change!

Signed-off-by: Steve Anderson <[email protected]>
Signed-off-by: Maximiliano Churichi <[email protected]>
@quintessence
Copy link
Contributor

@mchurichi I was looking at some of the PRs to see if I could help - is this still relevant or have things changed too much?

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.

Broken hyperlink in "Using Envoy with JWT-SVIDs"
7 participants