-
-
Notifications
You must be signed in to change notification settings - Fork 311
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: allow to configure keymanager token file path #6525
Conversation
@@ -93,12 +94,18 @@ export const keymanagerOptions: CliCommandOptions<KeymanagerArgs> = { | |||
default: false, | |||
group: "keymanager", | |||
}, | |||
"keymanager.authEnabled": { | |||
"keymanager.auth": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the flag to enable authentication to be more inline with other existing boolean flags
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably worth documenting in next release, so that people can migrate to the preferred syntax?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no need to migrate, existing setups can keep previous value
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6525 +/- ##
=========================================
Coverage 61.49% 61.49%
=========================================
Files 556 556
Lines 58895 58895
Branches 1856 1856
=========================================
Hits 36216 36216
Misses 22638 22638
Partials 41 41 |
Performance Report✔️ no performance regression detected Full benchmark results
|
@@ -93,12 +94,18 @@ export const keymanagerOptions: CliCommandOptions<KeymanagerArgs> = { | |||
default: false, | |||
group: "keymanager", | |||
}, | |||
"keymanager.authEnabled": { | |||
"keymanager.auth": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably worth documenting in next release, so that people can migrate to the preferred syntax?
@nflaig Is there a reason why this PR is in |
Waiting for spec PR discussion to resolve, might wanna rename flag name |
🎉 This PR is included in v1.18.0 🎉 |
Motivation
Description
Allow to configure keymanager token file path via
--keymanager.tokenFile /path/to/file
CLI flag