-
Notifications
You must be signed in to change notification settings - Fork 87
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
Upgrade Jetty from 9.4.5.v20170502 to 9.4.22.v20191022 #157
Conversation
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. A downstream PR would be great
To do what? |
hashLoginService.setUserStore(userStore); | ||
userStore.addUser("alice", new Password("alice"), new String[] {"user", "female"}); | ||
userStore.addUser("bob", new Password("bob"), new String[] {"user", "male"}); | ||
userStore.addUser("charlie", new Password("charlie"), new String[] {"user", "male"}); |
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 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.
I meant this Build Token Root plugin you referenced
Pointless since what I tested was behavior of |
Caused regression (#625) |
Supersedes #150. Picking up the sort of no-op servlet security realm that
JenkinsRule
uses. Confirmed that this works with thebuild-token-root
plugin with authentication enabled (initialAdminPassword
style).