Skip to content

Conversation

tpaschalis
Copy link
Member

Fixes #170

Signed-off-by: Paschalis Tsilias <[email protected]>
specification.md Outdated
### Configuration Management

* [Uber Flipr](https://eng.uber.com/flipr/)
* [Uber Flipr](https://web.archive.org/web/20240223111552/https://www.uber.com/blog/flipr/)
Copy link
Member Author

@tpaschalis tpaschalis Jul 1, 2024

Choose a reason for hiding this comment

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

The actual link is now https://www.uber.com/blog/flipr/, but hitting the link with cURL results in 406 Not Acceptable so I opted for the web.archive.org link.

Copy link
Contributor

Choose a reason for hiding this comment

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

Given that https://www.uber.com/blog/flipr exists, I'd vote to have an ignore pattern for this one.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ok, done in 515ba8a

### Security and Certificate Management

* [mTLS in Go](https://kofo.dev/how-to-mtls-in-golang)
* [mTLS in Go](https://web.archive.org/web/20240223142649/https://kofo.dev/how-to-mtls-in-golang)
Copy link
Member Author

Choose a reason for hiding this comment

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

This person's website has gone offline so I replaced this with the link from the Wayback Machine.

@tpaschalis tpaschalis marked this pull request as ready for review July 1, 2024 13:50
@tpaschalis tpaschalis requested a review from a team July 1, 2024 13:50
Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Thanks for figuring this out! See inline comment.

specification.md Outdated
### Configuration Management

* [Uber Flipr](https://eng.uber.com/flipr/)
* [Uber Flipr](https://web.archive.org/web/20240223111552/https://www.uber.com/blog/flipr/)
Copy link
Contributor

Choose a reason for hiding this comment

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

Given that https://www.uber.com/blog/flipr exists, I'd vote to have an ignore pattern for this one.

Signed-off-by: Paschalis Tsilias <[email protected]>
@tpaschalis tpaschalis requested a review from chalin July 1, 2024 15:28
@tigrannajaryan tigrannajaryan merged commit a3a0863 into open-telemetry:main Jul 2, 2024
@tigrannajaryan
Copy link
Member

Thank you @tpaschalis

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.

Markdown link checking isn't working
3 participants