Skip to content

Add fields key_usage and serial_number to vault_pki_secret_backend_intermediate_cert_request - #2404

Merged
victorr merged 4 commits into
mainfrom
victorr/vault-33909-intermediate-cert-request-key-usage
Feb 10, 2025
Merged

Add fields key_usage and serial_number to vault_pki_secret_backend_intermediate_cert_request#2404
victorr merged 4 commits into
mainfrom
victorr/vault-33909-intermediate-cert-request-key-usage

Conversation

@victorr

@victorr victorr commented Feb 10, 2025

Copy link
Copy Markdown
Contributor

Description

Add fields key_usage and serial_number to vault_pki_secret_backend_intermediate_cert_request

Checklist

  • Added CHANGELOG entry (only for user-facing changes)
  • Acceptance tests where run against all supported Vault Versions

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" comments, they generate extra noise for pull request followers and do not help prioritize the request

@victorr victorr self-assigned this Feb 10, 2025
@victorr
victorr marked this pull request as ready for review February 10, 2025 15:09
@victorr
victorr requested a review from a team as a code owner February 10, 2025 15:09
@victorr
victorr requested review from a team and erikapierr February 10, 2025 15:09
}

func testPkiSecretBackendIntermediateCertRequestConfig_key_usage(path string, optionalKeyUsage string) string {
var signatureBits string

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could we collapse all the various clones of this into a single function say testPkiSecretBackendIntermediateCertRequestConfig_generic, and have all these helper functions delegate the tweaked config to it? We have 3-4 copies of this terraform config now.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good idea, pushed a new changeset.

@stevendpclark stevendpclark left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@victorr
victorr merged commit f84ff08 into main Feb 10, 2025
@victorr
victorr deleted the victorr/vault-33909-intermediate-cert-request-key-usage branch February 10, 2025 16:02
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.

2 participants