Skip to content

Correct supported time definitions for self-destruct flags - #2250

Merged
sk-keeper merged 3 commits into
Keeper-Security:releasefrom
lthievenaz-keeper:fix-selfDestruct/help-dialog-correction
Jul 30, 2026
Merged

Correct supported time definitions for self-destruct flags#2250
sk-keeper merged 3 commits into
Keeper-Security:releasefrom
lthievenaz-keeper:fix-selfDestruct/help-dialog-correction

Conversation

@lthievenaz-keeper

Copy link
Copy Markdown
Contributor

In record-add and pam action rotate, the help menu gives incorrect time definitions for --self-destruct:
[(m)inutes|(h)ours|(d)ays]

Changed to:
[(mi)nutes|(h)ours|(d)ays|(mo)nths|(y)ears]

Before:
`[(m)inutes|(h)ours|(d)ays]`
After:
`[(mi)nutes|(h)ours|(d)ays|(mo)nths|(y)ears]`
Corrected the shorthand for minutes (`mi`, not `m`)
Added `mo`nths and `y`ears definitions
Before:
`[(m)inutes|(h)ours|(d)ays]`
After:
`[(mi)inutes|(h)ours|(d)ays|(mo)nths|(y)ears]`
@sk-keeper
sk-keeper merged commit 0964925 into Keeper-Security:release Jul 30, 2026
4 checks passed
sk-keeper pushed a commit that referenced this pull request Jul 30, 2026
* Correct the help dialog for Record Add --self-destruct

Before:
`[(m)inutes|(h)ours|(d)ays]`
After:
`[(mi)nutes|(h)ours|(d)ays|(mo)nths|(y)ears]`

* Correct self destruct time definitions

Corrected the shorthand for minutes (`mi`, not `m`)
Added `mo`nths and `y`ears definitions

* Correct the help dialog for PAM Rotate --self-destruct

Before:
`[(m)inutes|(h)ours|(d)ays]`
After:
`[(mi)inutes|(h)ours|(d)ays|(mo)nths|(y)ears]`
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