* Update MaskingClassLoader.java
Using the correct replace method
Modified the replace() with Character parameter for optimization
* Update LineEndingConversion.java
Using the correct replace method
Modified the replace(String, String) to replace(char, char) for optimization
* Update Icon.java
Using the correct replace method
Modified the replace() with Character parameter for optimization
* Admin monitors popup is now loaded through an API
* Fixes form submission inside the popup
- Wraps it in an l:ajax template so that proper variables are set
- Applies the csrf crumbs to the forms after the popup is constructed
* Handle authorization and empty state
* Apply suggestions from code review
Co-authored-by: Tim Jacomb <timjacomb1+github@gmail.com>
* Fix JS error where there were monitors of only one kind
* Revert changes to indentation of inline JS
* Clarify ESC keyCode with a comment
* Update core/src/main/resources/jenkins/management/AdministrativeMonitorsDecorator/resources.js
Co-authored-by: Tim Jacomb <timjacomb1+github@gmail.com>
* Multiple small adjustments
- Remove unused imports
- Remove non-necessary call for the response that are set in the getView
- Keep the previous improvement about limiting the number of call to getMonitorList()
Co-authored-by: Tim Jacomb <timjacomb1+github@gmail.com>
Co-authored-by: Wadeck Follonier <wadeck.follonier@gmail.com>
* Unforking XStream
* Some overrides missing from MapperDelegate
* [XSTR-762] writeReplace & readResolve must be accessible to be used from subclasses
* DomElement.click overloads changed incompatibly
* Giving up on ToolInstallation being Serializable.
Fixing MavenInstallation accordingly.
AntInstallation was already fixed: https://github.com/jenkinsci/ant-plugin/commit/c957dca4cd7789e3918e90ae020687aeaeea998e
* Relaxing message assertion in RobustReflectionConverter.randomExceptionsReported
* Picking up https://github.com/jenkinsci/matrix-auth-plugin/pull/89 caught via JobTest
* https://github.com/jenkinsci/jenkins-test-harness/pull/243 released
* Another case forgotten in 0e5aeec4cc
* XStream has SecurityMapper, but we have our own system already, so suppress warning
* GetMavenVersion was also serializing its outer class
* Trying to make ToolInstallation serializable compatible
* Avoid throwing IOException from PersistedList.inModified for common cases covered by https://github.com/jenkinsci/jenkins-test-harness/pull/243
* Found a way to avoid looking up XStream.converterRegistry by reflection
* XStream2Test.crashXstream now throwing ConversionException, not ForbiddenClassException
* Noting which version of the parent POM has https://github.com/jenkinsci/jenkins-test-harness/pull/243
* Refreshed Javadoc for XStream2
* The simplest solution to JENKINS-19561 seems to be to stop using autodetectAnnotations
* Fixing some deprecations in Robust*Converter
* Display OldDataMonitor errors from RobustReflectionConverter during functional tests
* Stray digit in test name
* Added XStream2AnnotationTest to clearly document incompatibility
* Picking up release of https://github.com/jenkinsci/matrix-auth-plugin/pull/89
Co-authored-by: Basil Crow <me@basilcrow.com>
Co-authored-by: Basil Crow <me@basilcrow.com>
The CaseInsensitive IdStrategy would always do a full string comparison
even when all that was required was an equivallence check.
updated the javadoc to clarify performance implications for subclasses
and fix the performance of the CaseInsensitive strategy
* RunList.subList: Reduce allocations in internal ArrayList
* LogRotator.perform: Remove copy of already new instance from subList
* LogRotator.perform: Change List to RunList for strict new List instance in builds.subList
* LogRotator: Remove unused imports
Co-authored-by: Ilya Gulyaev <ilya.gulyaev@nordigy.ru>
Basil Crow