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-71238] New login page breaks login-theme-plugin
#8341
[JENKINS-71238] New login page breaks login-theme-plugin
#8341
Conversation
If a fix in core is the way to go, any change it gets backported in next LTS ? |
While 2.414.1 is possible, yet a bit short-termed, 2.414.2 onwards would definitely be viable. |
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.
Thanks!
login-theme-plugin
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.
Looks OK security wise.
See JENKINS-71238.
This PR updates the login/register pages so that the 'header' element appears under the page title, as expected by its help text 'This HTML snippet will be inserted below the welcome message on the default login form'. Previously the text would appear on the left of the page.
Before (image from ticket)
After
I have left the 'head' element as is (as I'm assuming it's usage is just for
<head />
tags).Testing done
Proposed changelog entries
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
@jenkinsci/sig-ux
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).