-
Notifications
You must be signed in to change notification settings - Fork 48
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: Add Registry/Config Access token capability via Secret Provider #229
feat: Add Registry/Config Access token capability via Secret Provider #229
Conversation
Non-standard mixed case service key name change to simplified lower case `app-<profile name>` Removed previous file based access token config settings closes edgexfoundry#222 BREAKING CHANGE: When run with the secure Edgex Stack now need to have the SecretStore configured, a Vault token created and run with EDGEX_SECURITY_SECRET_STORE=true. BREAKING CHANGE: Service key changed to `app-<profile name>` Signed-off-by: lenny <leonard.goodell@intel.com>
The TAF tests will need to be update for the change in service key names. Thus this PR is in draft mode until those changes can be made. |
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.
some file-based leftover configuration to be removed
Signed-off-by: lenny <leonard.goodell@intel.com>
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.
LGTM
closes
BREAKING CHANGE: When run with the secure Edgex Stack now need to have the SecretStore configured, a Vault token created and run with EDGEX_SECURITY_SECRET_STORE=true.
BREAKING CHANGE: Service key changed to
app-<profile name>
Signed-off-by: lenny leonard.goodell@intel.com
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Registry/Config access tokens capability supported via files
Issue Number: #222
What is the new behavior?
Registry/Config access tokens capability now supported via SecretProvider
Non-standard mixed case service key name change to simplified lower case
app-<profile name>
Removed previous file based access token config settings
Does this PR introduce a breaking change?
BREAKING CHANGE: When run with the secure Edgex Stack now need to have the SecretStore configured, a Vault token created and run with EDGEX_SECURITY_SECRET_STORE=true.
BREAKING CHANGE: Service key changed to
app-<profile name>
Are there any new imports or modules? If so, what are they used for and why?
no
Are there any specific instructions or things that should be known prior to reviewing?
Other information