Skip to content

Conversation

sgmiller
Copy link
Contributor

System wide password policies now have a new config parameter,
entropy_source which allows customers to pick seal if they want entroy
from entropy augmentation.

@sgmiller sgmiller requested a review from a team as a code owner September 24, 2025 18:12
@github-actions github-actions bot added the Vault Content update for Vault product docs label Sep 24, 2025
Copy link

github-actions bot commented Sep 24, 2025

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Thu Sep 25 21:39:59 UTC 2025
Unified Docs API ✅ Ready (Inspect) Visit Preview Thu Sep 25 21:34:08 UTC 2025

Copy link

github-actions bot commented Sep 24, 2025

Broken Link Checker

No broken links found! 🎉

Comment on lines 43 to 46
- `entropy_source` `(string: "")` - Specifies the source of entropy
(randomness) used to generate the passwords. The default ("") and
"platform" source randomness from the platform RNG. "seal" (enterprise
only) sources entropy from the entropy augmentation.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- `entropy_source` `(string: "")` - Specifies the source of entropy
(randomness) used to generate the passwords. The default ("") and
"platform" source randomness from the platform RNG. "seal" (enterprise
only) sources entropy from the entropy augmentation.
- `entropy_source` `(string: "platform")` - Specifies the source of entropy
(randomness) used to generate the passwords. Must be one of:
- `platform` - source randomness from the platform RNG.
- `seal` - source entropy from the entropy augmentation. <EnterpriseAlert inline="true" />

It sounds like the default is platform?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The default is unset, because existing policies won't have a value for it. We could change the readout code to produce platform, but it'd produce Terraform diffs.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I like the formatting and ent fixes though

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Phrased it as "" means "default source"

@sgmiller sgmiller requested review from schavis and a team September 25, 2025 21:20
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