Skip to content

Vault 43670 update mssql doc for lease revocation#2374

Open
sachin-chand01 wants to merge 4 commits intovault/202605from
VAULT-43670-update-mssql-doc
Open

Vault 43670 update mssql doc for lease revocation#2374
sachin-chand01 wants to merge 4 commits intovault/202605from
VAULT-43670-update-mssql-doc

Conversation

@sachin-chand01
Copy link
Copy Markdown

PR Description

Summary

Updates the permissions for the MSSQL docs for this bug fix

MSSql Server database plugin throws "Logins other than the current user can only be seen by members of the sysadmin role." when using the default revocation statement.

Please go to the Preview tab and select the appropriate template:

Terraform

@sachin-chand01 sachin-chand01 requested a review from a team as a code owner May 5, 2026 12:42
@sachin-chand01 sachin-chand01 added the Vault Content update for Vault product docs label May 5, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Wed May 6 14:20:33 UTC 2026
Unified Docs API ✅ Ready (Inspect) Visit Preview Wed May 6 14:14:02 UTC 2026

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates Vault’s MSSQL database secrets engine documentation (across multiple maintained doc versions) to reflect a revocation behavior change that avoids requiring sysadmin, and adds a corresponding entry to the Vault 2.x release notes.

Changes:

  • Add required MSSQL permissions (VIEW ANY DEFINITION, VIEW SERVER STATE, and VIEW DEFINITION) to the “minimum functionality” setup snippets.
  • Document a limitation of the default revocation behavior when user creation occurs in a different database than the one targeted by connection_url.
  • Add a Vault 2.x release note describing the MSSQL revocation fix and reduced privilege requirements.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
content/vault/v2.x/content/docs/updates/release-notes.mdx Adds a Vault 2.x bug-fix release note for MSSQL revocation permissions.
content/vault/v2.x/content/docs/secrets/databases/mssql.mdx Updates MSSQL permission requirements and adds revocation behavior clarification.
content/vault/v1.21.x/content/docs/secrets/databases/mssql.mdx Backports the MSSQL permission + revocation behavior doc updates to 1.21 docs.
content/vault/v1.20.x/content/docs/secrets/databases/mssql.mdx Backports the MSSQL permission + revocation behavior doc updates to 1.20 docs.
content/vault/v1.19.x/content/docs/secrets/databases/mssql.mdx Backports the MSSQL permission + revocation behavior doc updates to 1.19 docs.

Comment thread content/vault/v2.x/content/docs/updates/release-notes.mdx
Comment thread content/vault/v2.x/content/docs/secrets/databases/mssql.mdx
Comment thread content/vault/v1.21.x/content/docs/secrets/databases/mssql.mdx
Comment thread content/vault/v1.20.x/content/docs/secrets/databases/mssql.mdx
Comment thread content/vault/v1.19.x/content/docs/secrets/databases/mssql.mdx
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

Broken Link Checker

This PR contains broken links, but won't be blocked. Use this report to improve content quality:

Quick Actions

  • Internal links (HashiCorp sites): Please fix these - they impact user experience
  • External links: Consider if these are essential or can be updated/removed
  • Temporary issues: External sites may recover - check again before merging

Need Help?


Internal Links

Full Github Actions output

External Links

Summary

Status Count
🔍 Total 39
✅ Successful 4
⏳ Timeouts 0
🔀 Redirected 4
👻 Excluded 31
❓ Unknown 0
🚫 Errors 0
⛔ Unsupported 0

Redirects per input

Redirects in content/vault/v1.19.x/content/docs/secrets/databases/mssql.mdx

Redirects in content/vault/v1.20.x/content/docs/secrets/databases/mssql.mdx

Redirects in content/vault/v1.21.x/content/docs/secrets/databases/mssql.mdx

Redirects in content/vault/v2.x/content/docs/secrets/databases/mssql.mdx

Full Github Actions output

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Comment thread content/vault/v2.x/content/docs/updates/release-notes.mdx
@sachin-chand01 sachin-chand01 changed the title Vault 43670 update mssql doc Vault 43670 update mssql doc for lease revocation May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Vault Content update for Vault product docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants