Skip to content

Conversation

@alexandrastroiu
Copy link

@alexandrastroiu alexandrastroiu commented Apr 26, 2025

Prerequisite Checklist

  • Read the contribution guidelines regarding submitting new changes to the project;
  • Tested your changes against relevant architectures and platforms;
  • Updated relevant documentation (if needed).

Description of changes

Replaced broken link referring to the CNS repository in chapters/mitigations-and-defensive-strategies/information-leaks/reading/README.md with the correct link.

Copy link
Contributor

@razvand razvand left a comment

Choose a reason for hiding this comment

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

Update the commit message line to info-leaks/reading: Fix broken ....

@razvand razvand mentioned this pull request May 10, 2025
3 tasks
@alexandrastroiu alexandrastroiu force-pushed the bugfix/fix-broken-link branch from 4211dfb to 111c0c8 Compare May 10, 2025 08:54
@alexandrastroiu alexandrastroiu requested a review from razvand May 10, 2025 09:06
@razvand razvand self-assigned this May 17, 2025
@alexandrastroiu alexandrastroiu requested a review from razvand May 17, 2025 11:09
Copy link
Contributor

@razvand razvand left a comment

Choose a reason for hiding this comment

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

Squash all commits into a single one.

The links referring to the CNS repository in the README.md file are
broken. This causes a 404 - page not found error. This commit fixes the
issue by replacing the broken links with the links to the precompiled
binaries in the CNS GitHub repository. Unnecessary link titles related to
the CNS repository were also removed.

Signed-off-by: Alexandra Stroiu <alexandra.stroiu@stud.acs.upb.ro>
@alexandrastroiu alexandrastroiu force-pushed the bugfix/fix-broken-link branch from 22934e2 to 5cfaab9 Compare May 17, 2025 19:55
@alexandrastroiu alexandrastroiu requested a review from razvand May 17, 2025 20:05
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.

2 participants