From 9ab50651fd972ead7184973ecb00daf75e090189 Mon Sep 17 00:00:00 2001 From: Devin Smith Date: Wed, 11 Oct 2023 17:58:46 -0700 Subject: [PATCH] Ensure that js-plugins aren't cached (#4621) This is an oversight discovered by web team and should have been part of #4389. Added #4620 as eventual follow-up. --- .../java/io/deephaven/server/jetty/JettyBackedGrpcServer.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/server/jetty/src/main/java/io/deephaven/server/jetty/JettyBackedGrpcServer.java b/server/jetty/src/main/java/io/deephaven/server/jetty/JettyBackedGrpcServer.java index 5a49ce3eb6c..f41aafb01f0 100644 --- a/server/jetty/src/main/java/io/deephaven/server/jetty/JettyBackedGrpcServer.java +++ b/server/jetty/src/main/java/io/deephaven/server/jetty/JettyBackedGrpcServer.java @@ -183,6 +183,8 @@ public JettyBackedGrpcServer( context.addFilter(new FilterHolder(filter), "/*", EnumSet.noneOf(DispatcherType.class)); // Wire up /js-plugins/* + // TODO(deephaven-core#4620): Add js-plugins version-aware caching + context.addFilter(NoCacheFilter.class, JS_PLUGINS_PATH_SPEC, EnumSet.noneOf(DispatcherType.class)); context.addServlet(servletHolder("js-plugins", jsPlugins.filesystem()), JS_PLUGINS_PATH_SPEC); // Set up websockets for grpc-web - depending on configuration, we can register both in case we encounter a