Merge pull request #35893 from vespa-engine/arnej/locale-model-evaluation

Make model-evaluation locale-independent

Author: arnej27959

model-evaluation/src/main/java/ai/vespa/models/handler/ModelsEvaluationHandler.java

@@ -103,7 +103,7 @@ private HttpResponse evaluateModel(HttpRequest request, Model model, String[] fu
             }
         }
         Tensor result = evaluator.evaluate();
-        return switch (property(request, "format.tensors").orElse("short").toLowerCase()) {
+        return switch (property(request, "format.tensors").orElse("short").toLowerCase(java.util.Locale.ROOT)) {
             case "short"        -> new Response(200, JsonFormat.encode(result, true,  false));
             case "long"         -> new Response(200, JsonFormat.encode(result, false, false));
             case "short-value"  -> new Response(200, JsonFormat.encode(result, true,  true));
@@ -205,7 +205,7 @@ public String[] range(int start, Optional<Integer> end) {
         }
 
         private static String[] splitPath(HttpRequest request) {
-            String path = request.getUri().getPath().toLowerCase();
+            String path = request.getUri().getPath().toLowerCase(java.util.Locale.ROOT);
             if (path.startsWith("/")) {
                 path = path.substring("/".length());
             }

model-evaluation/src/test/java/ai/vespa/models/handler/HandlerTester.java

@@ -130,7 +130,7 @@ void assertResponse(HttpRequest request, int expectedCode, Tensor expectedResult
     private String getContents(HttpResponse response) {
         try (ByteArrayOutputStream stream = new ByteArrayOutputStream()) {
             response.render(stream);
-            return stream.toString();
+            return stream.toString(StandardCharsets.UTF_8);
         } catch (IOException e) {
             throw new Error(e);
         }