Skip to content

Add autoconfirm policy as an exception to zero knowledge#820

Open
eliykat wants to merge 1 commit into
mainfrom
auto-confirm
Open

Add autoconfirm policy as an exception to zero knowledge#820
eliykat wants to merge 1 commit into
mainfrom
auto-confirm

Conversation

@eliykat

@eliykat eliykat commented Jun 19, 2026

Copy link
Copy Markdown
Member

🎟️ Tracking

Internal change

📔 Objective

Update Security Principle 01: Servers are zero knowledge to note the auto-confirm policy as an exception. This is already noted in our help documentation, but it's also relevant here because this says that it's a complete list of all exceptions.

📸 Screenshots

@eliykat eliykat requested review from a team as code owners June 19, 2026 03:33
@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR adds an "Automatic confirmation policy" section to the zero-knowledge security principle doc (01-servers-are-zero-knowledge.mdx), documenting it as a controlled exception. The new section follows the existing exception pattern (Key Connector, Icons service), uses correct heading capitalization and markdown link formatting, and accurately describes the security trade-off and its mitigations. No security, correctness, or breaking-change concerns were found.

Code Review Details

No actionable findings. This is a clean, documentation-only change consistent with the established structure and style of the surrounding content.

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