Skip to content

docs: add steps to list and delete public keys - #561

Merged
MantasMiksys merged 1 commit into
mainfrom
mm/delete-public-keys
Oct 8, 2026
Merged

MantasMiksys merged 1 commit into
mainfrom
mm/delete-public-keys

Conversation

@MantasMiksys

@MantasMiksys MantasMiksys commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Adds list and delete steps for public keys to the signed cookies guide, which only covered registering them.


Note

Low Risk
Documentation-only change with no runtime or security behavior impact.

Overview
Extends the signed-cookies guide with a List and delete public keys section so key lifecycle is documented beyond registration.

It documents aws cloudfront list-public-keys to discover key IDs (Id, Name, CreatedTime) and aws cloudfront delete-public-key --id … to remove a key, with a note that admin privileges are required for both operations.

Reviewed by Cursor Bugbot for commit 7b471ad. Bugbot is set up for automated code reviews on this repo. Configure here.

@vercel

vercel Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs-redirect Ready Ready Preview Oct 8, 2026 7:00am UTC
tigris-os-docs Ready Ready Preview Oct 8, 2026 7:00am UTC

Request Review

The signed cookies guide only showed how to register a public key. Add the
AWS CLI commands to list and delete them.
@greptile-apps

greptile-apps Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Low risk] Adds documentation steps for public key management.

The documentation change appears safe to merge.

Summary

Adds instructions to list and delete public keys in the signed-cookies guide.

  • Shows aws cloudfront list-public-keys and aws cloudfront delete-public-key.
  • Describes the listed fields and required admin privileges.
  • No new actionable issues were found.
  • MantasMiksys dismissed the previous deletion-warning suggestion with “Not needed.”

Reviews (2) · Last reviewed commit: "docs: add steps to list and delete publi..." · Reviewed by Greptile

Comment thread docs/objects/access-objects-via-cookies.md
@MantasMiksys

Copy link
Copy Markdown
Contributor Author

@greptile review

@MantasMiksys
MantasMiksys merged commit 0843b0e into main Oct 8, 2026
12 checks passed
@MantasMiksys
MantasMiksys deleted the mm/delete-public-keys branch October 8, 2026 07:23

This branch was successfully deployed

2 active deployments
Preview – tigris-os-docs — 7b471adb Deployed Oct 8, 2026 by vercel[bot]
Preview – docs-redirect — 7b471adb Deployed Oct 8, 2026 by vercel[bot]
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.

1 participant