Skip to content

change token type ttl to string#34

Merged
chris199512 merged 1 commit intomasterfrom
fix-type
Sep 15, 2025
Merged

change token type ttl to string#34
chris199512 merged 1 commit intomasterfrom
fix-type

Conversation

@nefeldt
Copy link
Copy Markdown
Contributor

@nefeldt nefeldt commented Sep 15, 2025

https://developer.hashicorp.com/vault/api-docs/auth/token#create-token
ttl should be string not int

ttl (string: "") - The TTL period of the token, provided as "1h", where hour is the largest suffix. If not provided, the token is valid for the default lease TTL, or indefinitely if the root policy is used.

@chris199512 chris199512 merged commit a1fe696 into master Sep 15, 2025
3 checks passed
@chris199512 chris199512 requested a review from Copilot September 15, 2025 13:12
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 aligns the TTL field type in the authentication token creation request with the HashiCorp Vault API specification, which requires TTL to be a string (e.g., "1h") rather than an integer.

  • Changes the TTL field type from int to string in the AuthCreateTokenRequest struct

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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.

3 participants