GeoServerPropertyOverrideConfigurer doesn't properly substitute windows paths

Description

GeoServerPropertyOverrideConfigurer uses replaceAll() method to substitute GEOSERVER_DATA_DIR placeholders with GeoServer data dir path. Problem, the replaceAll() method uses regex instead of literal strings, so when substituting with a windows path this will produce a wrong result.

For example, if GeoServer data dir path is

and we have a property like this

the substitution result will be:

Using replace() method instead of replaceAll() method will produced the expected result:

Environment

None

Activity

Show:
Nuno Oliveira
May 8, 2016, 9:08 PM
Edited

Here are the links to the pull requests (including backport to 2.9.x) that fix this issue:

Fixed

Assignee

Nuno Oliveira

Reporter

Nuno Oliveira

Fix versions