* JENKINS-13128: Preserve copied file permissions and mtime
This fixes a bug where files copied locally do not preserve file permissions or last modification time.
* Use IO utility methods for exception handling
* Fix invalid path exception propagation
* Revert hudson.Util
* JENKINS-50412 - Fingerprints logs should be less verbose
Eliminate the logging line that says
possibly trimming /var/jenkins_home/fingerprints/[...]
This line fills up the fingerprints log file with repetitive, useless information.
The operation it calls provides its own logging that can be turned on if detailed
logging is actually needed instead of running this one repeatedly.
Also, while in the area, convert a couple of FileFilter inner classes to lambdas
and inline them. This reduces unnecessary boilerplate and improves clarity.
In order to make this change testable, I elected to remove the final restriction on the
class and annotate the class as restricted. This allows me to create unit tests for
this change and also verify existing capability at the appropriate level.
* Watch out for varying line endings.
Use a reliable check.
* Correct a few minor issues caught in review.
* Change to correct access modifier.
* [JENKINS-9104] Add a test method to show that vetos aren't processed on slaves
* Move the veto check to always run on the master
* Add a flag to skip master call if empty list
The special old module for doxia seems not needed.
I suspect that it is because since 2012 when it was released, the Markdown
support got done in standard.
IIUC this was pulling in some very old doxia versions, which was
incompatible with maven-fluido-skin which I needed to bump from current
1.3.0 to 1.7 for https://github.com/jenkins-infra/core-taglibs-report-generator
to work again.
* [JENKINS-31661] Add an administrative monitor to check the root url configuration
- add the monitor + test
* - corrections requested by Oleg in code review
* - add a field to fill during SetupWizard for the rootUrl (default value using javascript but editable)
- WIP : missing the translation
* - revert whitespace change
* - little refactoring of js methods
* - translation added
* - modification requested during PR by Jesse
* - renaming proposed by Keith
* - rework to have a new page in the SetupWizard for the root URL
* Correct relevant non-conflicting merge changes
* - in case of skip => also skip the instance configuration
- terminate => finish (+ uppercase)
- replace double single quotes
- add help message in the wizard to inform admins
- correct the installState serialization problem
- update working (admin monitor + wizard)
* address Daniel's comment
- skip all steps will show two messages instead of one (like before when clicking on skip twice)
- skip => not now
- remove margin and adjust padding for the wizard forms
- add the "if possible"
* address Arnaud's comment on the sauter vs passer
* address Liam's comment
* correct previous commit typo
* correct previous commit typo (2)
* add blank end line
* address James' comment
* address Daniel's comment
* address Devin's comment
- implement error management in the wizard
- provide different message in the monitor in case of null / invalid url
- add an helper to validate URL (even with _ in domain)
- add @Restricted
* - also update the monitor test to be consistent with its new behavior
* address Devin's comments
* correct header files
* include relevant classes for the url validation from commons-validator:1.6
* Move Apache classes in a more standard package
Oleg Nenashev
Jesse Glick
Kohsuke Kawaguchi
Daniel Beck
Hangsu Ma
aviadatsnyk
Wadeck Follonier
Matt Sicker
Jeff Thompson
Devin Nusbaum
Baptiste Mathus
Baptiste Mathus
Andy Neebel
Jochen Fürbacher
Oliver Gondža
Wadeck Follonier