Skip to content
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

Second backporting for 2.462.2 #9631

Merged
merged 3 commits into from
Aug 20, 2024

Conversation

NotMyFault
Copy link
Member

Latest core version: jenkins-2.472

Fixed

[JENKINS-73622] Minor 2.467
Backport Spring Framework 5.3.39 into 2.462.2
https://issues.jenkins.io/browse/JENKINS-73622

[JENKINS-73404] Minor 2.469
SecretTextArea missing FormValidation support
https://issues.jenkins.io/browse/JENKINS-73404

jtnord and others added 2 commits August 19, 2024 17:20
…sci#9450)

* [JENKINS-73404] unit test

* roundtrip as Secrets

* [JENKINS-73404] add validation for SecretTextArea

reworked the jelly and simplified the javascript.

the textarea is now always present in the DOM, so no need to remove
inputs to add a textarea.  Rather we just remove /hide the "hidden" attribute
from the elements we want to show/hide and give the textArea the
expected class and attributes for validation to occur

* remove unused data-prompt attribute

* Adapt getHiddenSecretValue to backend changes

* Adapt to Secrets being sent as "" rather than null

values for SecretTextArea are now always sent.  Whilst this is a change
in behavior the only way once set to unset is to set the value to the
mpty string "" which would still produce a Secret

Additionally this matches the behaviour of f:password and f:textarea

Given both of the above this change in behaviour should not break any
production use cases.

* Remove trailing spaces

---------

Co-authored-by: Mark Waite <mark.earl.waite@gmail.com>
(cherry picked from commit 81b90e4)
…jenkinsci#9612)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
(cherry picked from commit c5c69ab)
@NotMyFault NotMyFault added the into-lts This PR is filed against an LTS branch label Aug 19, 2024
Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

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

Thanks!

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
(cherry picked from commit aa348fc)
@basil basil merged commit 91b4d80 into jenkinsci:stable-2.462 Aug 20, 2024
17 checks passed
@NotMyFault NotMyFault deleted the 2nd-backporting-2.462.2 branch August 20, 2024 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
into-lts This PR is filed against an LTS branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants