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
[JENKINS-65398] terminology update #197
Conversation
src/main/java/com/cloudbees/plugins/credentials/CredentialsProviderFilter.java
Outdated
Show resolved
Hide resolved
src/main/resources/com/cloudbees/plugins/credentials/BaseCredentials/help-scope.html
Outdated
Show resolved
Hide resolved
…ntials/help-scope.html
...redentials/impl/CertificateCredentialsImpl/FileOnMasterKeyStoreSource/help-keyStoreFile.html
Outdated
Show resolved
Hide resolved
…ificateCredentialsImpl/FileOnMasterKeyStoreSource/help-keyStoreFile.html
@@ -52,7 +52,7 @@ While the Credentials API plugin provides a default _internal_ credentials store | |||
If you are using the _internal_ credentials store to store *high value* credentials then you will need to lock down your Jenkins configuration: | |||
|
|||
* Apply all of the recommendations from the https://wiki.jenkins.io/display/JENKINS/Securing+Jenkins[Securing Jenkins] wiki page. | |||
* No builds on the master (ideally zero executors) | |||
* No builds on the controller (ideally zero executors) |
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.
Potentially wrong term
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.
What would you suggest?
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.
Still undefined due to lack of followup for the last 8 months.
Removing offending terminology such as master,slave or whitelist/blacklist as part of JENKINS-65398
I tried to update whitelist/ black list with the context, feel free to tell me if it's not accurate.