Skip to content

Add a crates.io-specific security page #8791

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

Merged
merged 5 commits into from
Jun 10, 2024

Conversation

carols10cents
Copy link
Member

The security response WG feels the ecosystem help communication is better suited to a crates.io-specific security page, so this creates one with that information and some other information that is relevant to crates.io but not really relevant on the Rust security page.

This PR:

  • Creates crates.io/security
  • Moves some security-related content from crates.io/policies to this page (but leaves the header with the ID and a link to the security page to avoid breaking any links to that heading)
  • Changes the "Security" link in the footer to go to this new page

Once this PR is merged and deployed, follow-up work will include:

Background references:

@carols10cents carols10cents force-pushed the our-very-own-security-policy branch from 3fa75ca to 272dd9b Compare June 4, 2024 18:31
@carols10cents carols10cents requested a review from LawnGnome June 4, 2024 18:31
Copy link
Member

@pietroalbini pietroalbini left a comment

Choose a reason for hiding this comment

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

The page looks awesome, thanks Carol! Left just a small comment.

And (most) other crates hosted on crates.io are not
Copy link
Member

@pietroalbini pietroalbini left a comment

Choose a reason for hiding this comment

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

Looks good on my end!

Copy link
Member

@Manishearth Manishearth left a comment

Choose a reason for hiding this comment

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

LG from my end as well

@Turbo87 Turbo87 added A-frontend 🐹 C-documentation 📜 Category: Documentation of crates.io itself labels Jun 6, 2024
@carols10cents carols10cents force-pushed the our-very-own-security-policy branch from 7370077 to 1283c54 Compare June 6, 2024 13:46
Copy link
Member

@cuviper cuviper left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@Turbo87 Turbo87 merged commit a35ec8f into rust-lang:main Jun 10, 2024
8 checks passed
@carols10cents carols10cents deleted the our-very-own-security-policy branch June 12, 2024 14:23
carols10cents added a commit that referenced this pull request Jul 9, 2024
Follow-up work to #8791 now
that the page is live.
carols10cents added a commit that referenced this pull request Jul 11, 2024
Follow-up work to #8791 now
that the page is live.
Turbo87 pushed a commit to Turbo87/crates.io that referenced this pull request Jul 12, 2024
Turbo87 added a commit that referenced this pull request Jul 12, 2024
Follow-up work to #8791 now
that the page is live.

Co-authored-by: Carol (Nichols || Goulding) <carol.nichols@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-frontend 🐹 C-documentation 📜 Category: Documentation of crates.io itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants