diff options
Diffstat (limited to 'config-application-package/src/main/java/com/yahoo')
-rw-r--r-- | config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java b/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java index 5ec30f71e7b..b2a1507b424 100644 --- a/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java +++ b/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java @@ -609,13 +609,11 @@ public class FilesApplicationPackage implements ApplicationPackage { /** * Reads a ranking expression from file to a string and returns it. * - * @param name the name of the file to return, either absolute or + * @param name the name of the file to return, * relative to the search definition directory in the application package * @return the content of a ranking expression file * @throws IllegalArgumentException if the file was not found or could not be read */ - // TODO: A note on absolute paths: We don't want to support this and it should be removed on 6.0 - // Currently one system test (basicmlr) depends on it. @Override public Reader getRankingExpression(String name) { try { @@ -628,8 +626,9 @@ public class FilesApplicationPackage implements ApplicationPackage { private File expressionFileNameToFile(String name) { File expressionFile = new File(name); - if (expressionFile.isAbsolute()) return expressionFile; - + if (expressionFile.isAbsolute()) { + throw new IllegalArgumentException("Absolute path to ranking expression file is not allowed: " + name); + } File sdDir = new File(appDir, ApplicationPackage.SEARCH_DEFINITIONS_DIR.getRelative()); return new File(sdDir, name); } |