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-18884] Add People View #9052
[JENKINS-18884] Add People View #9052
Conversation
Please take a moment and address the merge conflicts of your pull request. Thanks! |
5c63b72
to
2f5c5c1
Compare
Previously: #1094 and #1102 (which this resurrects almost exactly). Could you explain what this PR is supposed to do? I am asking because it doesn't actually do what the title says, and likely not what at least a subset of people in the linked issue need. Are there reasons you decided against implementing the suggestions from #1102 (comment) / #1102 (comment) / #1102 (comment) / #1102 (comment)? FWIW I would currently prefer a removal from core (no implicit dependency) with simultaneous release of a replacement plugin, similar to https://plugins.jenkins.io/cctray-xml/ |
basically I just tried to get past work done and resolve the mentioned issue. For extracting to a new plugin somebody else has to takeover, i can not handle that. |
FTR #9060 but note that it may not work out (or may not be a consensus preferred approach). |
Closing this pull request since it has been superseded by Thanks very much @hypery2k |
See JENKINS-18884. And People page of All view also displays all users registered in Jenkins.
Testing done
Proposed changelog entries
People page of All view no longer display all users registered in Jenkins. It works same as those of other views:
There is no reason for People page of All page to work different from those of other views.
Displaying all users in Jenkins should be handled by security realms (like HudsonPrivateSecurityRealm.ManageUserLinks).
This is another approach of [JENKINS-18884] Added configuration for access to "People" page. #1094 .
Proposed upgrade guidelines
N/A
Submitter checklist
@Restricted
or have@since TODO
Javadocs, as appropriate.@Deprecated(since = "TODO")
or@Deprecated(forRemoval = true, since = "TODO")
, if applicable.eval
to ease future introduction of Content Security Policy (CSP) directives (see documentation).Desired reviewers
@mention
Before the changes are marked as
ready-for-merge
:Maintainer checklist
upgrade-guide-needed
label is set and there is a Proposed upgrade guidelines section in the pull request title (see example).lts-candidate
to be considered (see query).