Skip to content

[JENKINS-61131] Fix broken credentials selection#116

Merged
pjdarton merged 1 commit intojenkinsci:masterfrom
albers:fix-jenkins-61131
Mar 9, 2020
Merged

[JENKINS-61131] Fix broken credentials selection#116
pjdarton merged 1 commit intojenkinsci:masterfrom
albers:fix-jenkins-61131

Conversation

@albers
Copy link
Copy Markdown
Member

@albers albers commented Mar 3, 2020

Fixes JENKINS-61131.

#110 changed the population of the credentials selection to no longer use deprecated api.
Credentials are now resolved in the context of Jenkins.getAuthentication(), which does not return any matches.

According to the documentation of the credentials-plugin, interactive lookups on the main
configuration page should be performed as ACL.SYSTEM.

Signed-off-by: Harald Albers github@albersweb.de

jenkinsci#110 changed the population of the credentials selection to no longer use deprecated api.
Credentials are now resolved in the context of Jenkins.getAuthentication(), which does not return any matches.

According to the documentation of the credentials-plugin, interactive lookups on the main
configuration page should be performed as ACL.SYSTEM.

Signed-off-by: Harald Albers <github@albersweb.de>
Copy link
Copy Markdown
Member

@jetersen jetersen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pjdarton pjdarton merged commit 36166a4 into jenkinsci:master Mar 9, 2020
@pjdarton
Copy link
Copy Markdown
Member

pjdarton commented Mar 9, 2020

Thanks for the fix; much appreciated.

@albers albers deleted the fix-jenkins-61131 branch March 10, 2020 19:37
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