Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5665#discussion_r181775179
--- Diff:
flink-runtime-web/src/test/java/org/apache/flink/runtime/webmonitor/WebFrontendITCase.java
---
@@ -127,14 +137,18 @@ public void testResponseHeaders() throws Exception {
Assert.assertEquals("application/json; charset=UTF-8",
taskManagerConnection.getContentType());
// check headers in case of an error
- URL notFoundJobUrl = new URL("http://localhost:"; + port +
"/jobs/dontexist");
+ URL notFoundJobUrl = new URL("http://localhost:"; +
CLUSTER.getWebUIPort() + "/jobs/dontexist");
HttpURLConnection notFoundJobConnection = (HttpURLConnection)
notFoundJobUrl.openConnection();
notFoundJobConnection.setConnectTimeout(100000);
notFoundJobConnection.connect();
if (notFoundJobConnection.getResponseCode() >= 400) {
// we don't set the content-encoding header
Assert.assertNull(notFoundJobConnection.getContentEncoding());
- Assert.assertEquals("text/plain; charset=UTF-8",
notFoundJobConnection.getContentType());
+ if (Objects.equals("flip6",
System.getProperty("codebase"))) {
--- End diff --
this must be updated to "new"
---
