From fb9d876bff2e1fdaed88c9dba3910ebf90269c2a Mon Sep 17 00:00:00 2001
From: mcilwain <mcilwain@google.com>
Date: Tue, 26 Jun 2018 18:03:40 -0700
Subject: [PATCH] Remove most HTML/CSS/JS assets from the backend service WAR

It only needs the error page HTML files; everything else isn't used by endpoints
served by the backend service and only serves to increase build times
(especially compiling all that JS).

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=202229514
---
 java/google/registry/BUILD                       |  2 +-
 .../env/alpha/backend/WEB-INF/appengine-web.xml  |  3 ---
 .../env/crash/backend/WEB-INF/appengine-web.xml  |  3 ---
 .../env/local/backend/WEB-INF/appengine-web.xml  |  9 ---------
 .../production/backend/WEB-INF/appengine-web.xml |  3 ---
 .../sandbox/backend/WEB-INF/appengine-web.xml    |  3 ---
 java/google/registry/ui/BUILD                    | 16 ++++++++++++++++
 7 files changed, 17 insertions(+), 22 deletions(-)

diff --git a/java/google/registry/BUILD b/java/google/registry/BUILD
index c1ff37f8a..815c5ebec 100644
--- a/java/google/registry/BUILD
+++ b/java/google/registry/BUILD
@@ -133,7 +133,7 @@ zip_file(
     },
     deps = [
         ":common_war",
-        "//java/google/registry/ui:war",
+        "//java/google/registry/ui:minimal_war",
     ],
 )
 
diff --git a/java/google/registry/env/alpha/backend/WEB-INF/appengine-web.xml b/java/google/registry/env/alpha/backend/WEB-INF/appengine-web.xml
index 54f782c31..9d026e1ce 100644
--- a/java/google/registry/env/alpha/backend/WEB-INF/appengine-web.xml
+++ b/java/google/registry/env/alpha/backend/WEB-INF/appengine-web.xml
@@ -22,8 +22,5 @@
 
   <static-files>
     <include path="/*.html" expiration="1m"/>
-    <include path="/assets/js/**" expiration="1m"/>
-    <include path="/assets/css/**" expiration="1m"/>
-    <include path="/assets/images/**" expiration="1m"/>
   </static-files>
 </appengine-web-app>
diff --git a/java/google/registry/env/crash/backend/WEB-INF/appengine-web.xml b/java/google/registry/env/crash/backend/WEB-INF/appengine-web.xml
index b6f8925b3..47aa68a9a 100644
--- a/java/google/registry/env/crash/backend/WEB-INF/appengine-web.xml
+++ b/java/google/registry/env/crash/backend/WEB-INF/appengine-web.xml
@@ -22,8 +22,5 @@
 
   <static-files>
     <include path="/*.html" expiration="1m"/>
-    <include path="/assets/js/**" expiration="1m"/>
-    <include path="/assets/css/**" expiration="1m"/>
-    <include path="/assets/images/**" expiration="1m"/>
   </static-files>
 </appengine-web-app>
diff --git a/java/google/registry/env/local/backend/WEB-INF/appengine-web.xml b/java/google/registry/env/local/backend/WEB-INF/appengine-web.xml
index a5e451cf2..dc2f284d1 100644
--- a/java/google/registry/env/local/backend/WEB-INF/appengine-web.xml
+++ b/java/google/registry/env/local/backend/WEB-INF/appengine-web.xml
@@ -26,14 +26,5 @@
     <include path="/*.html">
       <http-header name="Cache-Control" value="max-age=0,must-revalidate" />
     </include>
-    <include path="/assets/js/**">
-      <http-header name="Cache-Control" value="max-age=0,must-revalidate" />
-    </include>
-    <include path="/assets/css/**">
-      <http-header name="Cache-Control" value="max-age=0,must-revalidate" />
-    </include>
-    <include path="/assets/images/**">
-      <http-header name="Cache-Control" value="max-age=0,must-revalidate" />
-    </include>
   </static-files>
 </appengine-web-app>
diff --git a/java/google/registry/env/production/backend/WEB-INF/appengine-web.xml b/java/google/registry/env/production/backend/WEB-INF/appengine-web.xml
index e99978109..244c1f043 100644
--- a/java/google/registry/env/production/backend/WEB-INF/appengine-web.xml
+++ b/java/google/registry/env/production/backend/WEB-INF/appengine-web.xml
@@ -22,9 +22,6 @@
 
   <static-files>
     <include path="/*.html" expiration="1d"/>
-    <include path="/assets/js/**" expiration="1d"/>
-    <include path="/assets/css/**" expiration="1d"/>
-    <include path="/assets/images/**" expiration="1d"/>
   </static-files>
 
   <!-- Prevent uncaught servlet errors from leaking a stack trace. -->
diff --git a/java/google/registry/env/sandbox/backend/WEB-INF/appengine-web.xml b/java/google/registry/env/sandbox/backend/WEB-INF/appengine-web.xml
index 69f1f6773..16cb0dd71 100644
--- a/java/google/registry/env/sandbox/backend/WEB-INF/appengine-web.xml
+++ b/java/google/registry/env/sandbox/backend/WEB-INF/appengine-web.xml
@@ -22,9 +22,6 @@
 
   <static-files>
     <include path="/*.html" expiration="1d"/>
-    <include path="/assets/js/**" expiration="1d"/>
-    <include path="/assets/css/**" expiration="1d"/>
-    <include path="/assets/images/**" expiration="1d"/>
   </static-files>
 
   <!-- Prevent uncaught servlet errors from leaking a stack trace. -->
diff --git a/java/google/registry/ui/BUILD b/java/google/registry/ui/BUILD
index 00b0a38e0..911079fc6 100644
--- a/java/google/registry/ui/BUILD
+++ b/java/google/registry/ui/BUILD
@@ -17,6 +17,13 @@ filegroup(
     ],
 )
 
+filegroup(
+    name = "minimal_runfiles",
+    srcs = [
+        "//java/google/registry/ui/html:html_files",
+    ],
+)
+
 filegroup(
     name = "runfiles_debug",
     srcs = [
@@ -41,6 +48,15 @@ zip_file(
     },
 )
 
+zip_file(
+    name = "minimal_war",
+    srcs = [":minimal_runfiles"],
+    out = "minimal_ui.war",
+    mappings = {
+        "domain_registry/java/google/registry/ui/html": "",
+    },
+)
+
 zip_file(
     name = "war_debug",
     srcs = [":runfiles_debug"],