Skip to content

docs: fix security links in README - #5452

Open
kjanat wants to merge 1 commit into
opencontainers:mainfrom
kjanat:fix/docs-security-links
Open

docs: fix security links in README#5452
kjanat wants to merge 1 commit into
opencontainers:mainfrom
kjanat:fix/docs-security-links

Conversation

@kjanat

@kjanat kjanat commented Sep 8, 2026

Copy link
Copy Markdown

The README's security policy and audit-report links still reference the former master branch. GitHub currently redirects those links to main.

Update both links to reference main directly.

Earlier audit-report link screenshot Screenshot From 2026-09-08 21-25-54

P.S. Thx for using kjanat/actionlint!

@kjanat
kjanat force-pushed the fix/docs-security-links branch 2 times, most recently from 6a28bf7 to 6f24f89 Compare September 8, 2026 19:56
@AkihiroSuda

Copy link
Copy Markdown
Member

I encountered a 404 when following the audit-report link, as shown below.

https://github.com/opencontainers/runc/blob/master/docs/Security-Audit.pdf works for me (automatically redirected to https://github.com/opencontainers/runc/blob/main/docs/Security-Audit.pdf)

@kolyshkin kolyshkin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hmm the old links still work for me (github redirects those to main).

@kolyshkin

Copy link
Copy Markdown
Contributor

so maybe we can just s/master/main/g here?

The README's security policy and audit-report links still reference the
former `master` branch. GitHub currently redirects those links to
`main`.

Update both links to reference `main` directly.

Signed-off-by: Kaj Kowalski <[email protected]>
@kjanat
kjanat force-pushed the fix/docs-security-links branch from 6f24f89 to 1c45514 Compare September 9, 2026 00:10
@kjanat

kjanat commented Sep 9, 2026

Copy link
Copy Markdown
Author

Sure, makes sense. I've updated both branch references to main.

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