sanitize more paths on windows, fixes #2335

[tarzanek]

this is a good start, next steps are sanitize C:\ or any other letter and strip them
and make sure whole app just uses "/" and on windows all input or output gets properly sanitized and stripped if needed

[coveralls]

Pull Request Test Coverage Report for Build 3483

• 27 of 32 (84.38%) changed or added relevant lines in 7 files are covered.
• 4 unchanged lines in 2 files lost coverage.
• Overall coverage increased (+0.001%) to 73.499%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
opengrok-indexer/src/main/java/org/opengrok/indexer/web/Util.java	13	14	92.86%
opengrok-indexer/src/main/java/org/opengrok/indexer/web/PageConfig.java	7	11	63.64%

Files with Coverage Reduction	New Missed Lines	%
opengrok-indexer/src/main/java/org/opengrok/indexer/history/HistoryGuru.java	2	66.94%
opengrok-indexer/src/main/java/org/opengrok/indexer/history/BazaarRepository.java	2	70.16%

Totals
Change from base Build 3476:	0.001%
Covered Lines:	32105
Relevant Lines:	43681

---

💛 - Coveralls

[tarzanek]

need to backout the executable changes in tools/pom.xml

[tulinkry]

this does not lead to the local build and installs the package to the global scope

[tulinkry]

conditional rule?

[tulinkry]

the first letter should be lowercase

[tulinkry]

or if its constant, it should be final and all upper case

[tulinkry]

you sure that there will be no npe?

[tulinkry]

some reformat would help here

[tulinkry]

property should be on top of the class

[tulinkry]

what if user has the data on D: ?

[tarzanek]

so besides addressing the data on d: these changes should work, let's see about tests