[FLINK-4304] [runtime-web] Jar names that contain whitespace cause pr…

…oblems in web client

This closes apache#2327.

flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/RuntimeMonitorHandler.java

@@ -29,6 +29,7 @@
 import io.netty.handler.codec.http.router.KeepAliveWrite;
 import io.netty.handler.codec.http.router.Routed;
 
+import java.net.URLDecoder;
 import org.apache.flink.runtime.instance.ActorGateway;
 import org.apache.flink.runtime.webmonitor.handlers.RequestHandler;
 import org.apache.flink.util.ExceptionUtils;
@@ -82,10 +83,15 @@ protected void respondAsLeader(ChannelHandlerContext ctx, Routed routed, ActorGa
 				queryParams.put(key, routed.queryParam(key));
 			}
 
+			Map<String, String> pathParams = new HashMap<>(routed.pathParams().size());
+			for (String key : routed.pathParams().keySet()) {
+				pathParams.put(key, URLDecoder.decode(routed.pathParams().get(key), ENCODING.toString()));
+			}
+
 			InetSocketAddress address = (InetSocketAddress) ctx.channel().localAddress();
 			queryParams.put(WEB_MONITOR_ADDRESS_KEY, address.getHostName() + ":" + address.getPort());
 
-			String result = handler.handleRequest(routed.pathParams(), queryParams, jobManager);
+			String result = handler.handleRequest(pathParams, queryParams, jobManager);
 			byte[] bytes = result.getBytes(ENCODING);
 
 			response = new DefaultFullHttpResponse(

flink-runtime-web/web-dashboard/app/scripts/modules/submit/submit.svc.coffee

@@ -32,7 +32,7 @@ angular.module('flinkApp')
   @deleteJar = (id) ->
     deferred = $q.defer()
 
-    $http.delete("jars/" + id)
+    $http.delete("jars/" + encodeURIComponent(id))
     .success (data, status, headers, config) ->
        deferred.resolve(data)
 
@@ -41,7 +41,7 @@ angular.module('flinkApp')
   @getPlan = (id, args) ->
     deferred = $q.defer()
 
-    $http.get("jars/" + id + "/plan", {params: args})
+    $http.get("jars/" + encodeURIComponent(id) + "/plan", {params: args})
     .success (data, status, headers, config) ->
       deferred.resolve(data)
 
@@ -50,7 +50,7 @@ angular.module('flinkApp')
   @runJob = (id, args) ->
     deferred = $q.defer()
 
-    $http.post("jars/" + id + "/run", {}, {params: args})
+    $http.post("jars/" + encodeURIComponent(id) + "/run", {}, {params: args})
     .success (data, status, headers, config) ->
       deferred.resolve(data)