Skip to content

Update README.md: link to ostif report#6809

Closed
alexbozhenko wants to merge 2 commits intomainfrom
alexbozhenko-patch-3
Closed

Update README.md: link to ostif report#6809
alexbozhenko wants to merge 2 commits intomainfrom
alexbozhenko-patch-3

Conversation

@alexbozhenko
Copy link
Member

@alexbozhenko alexbozhenko commented Apr 18, 2025

Signed-off-by: Alex Bozhenko alex@synadia.com

@alexbozhenko alexbozhenko requested a review from a team as a code owner April 18, 2025 17:50
README.md Outdated

A third party security audit was performed by Cure53, you can see the full report [here](https://github.com/nats-io/nats-general/blob/main/reports/Cure53_NATS_Audit.pdf).
A third-party security audit was performed by ostif.org with the help of "Trail of Bits".
You can see the full report [on Trail of Bit’s Github](https://github.com/trailofbits/publications/blob/master/reviews/2025-04-ostif-nats-securityreview.pdf).
Copy link
Member

Choose a reason for hiding this comment

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

"Trail of Bits GitHub", no apostrophe. Can just change the link to "here"?

Copy link
Member Author

Choose a reason for hiding this comment

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

I personally like longer links because I think 4 chars are easier to miss, but I can change if you insist :-)

Copy link
Member

Choose a reason for hiding this comment

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

I suggest linking to the blog post and the report directly. A compromise would be be two links that are named and have purpose.

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.

3 participants