Support moving static ode to another server

This permits serving the static portions of ode (the Javascript code and translations) from another web server, including via a third party CDN such as Cloudflare. Change-Id: Ic70ca21419c67be66e5c9011c42d53bafe9367bf
mit-cml · Mar 13, 2024 · 298e589 · 298e589
1 parent 9c94514
commit 298e589
Show file tree

Hide file tree

Showing 13 changed files with 89 additions and 12 deletions.
diff --git a/appinventor/appengine/src/com/google/appinventor/client/Ode.java b/appinventor/appengine/src/com/google/appinventor/client/Ode.java
@@ -89,6 +89,7 @@
 import com.google.gwt.user.client.Timer;
 import com.google.gwt.user.client.Window;
 import com.google.gwt.user.client.rpc.AsyncCallback;
+import com.google.gwt.user.client.rpc.ServiceDefTarget;
 import com.google.gwt.user.client.rpc.StatusCodeException;
 import com.google.gwt.user.client.ui.Button;
 import com.google.gwt.user.client.ui.CheckBox;
@@ -684,6 +685,14 @@ public void onUncaughtException(Throwable e) {
     // This call also stores our sessionId in the backend. This will be checked
     // when we go to save a file and if different file saving will be disabled
     // Newer sessions invalidate older sessions.
+
+    setupOrigin(((ServiceDefTarget)projectService));
+    setupOrigin((ServiceDefTarget)userInfoService);
+    setupOrigin((ServiceDefTarget)getMotdService);
+    setupOrigin((ServiceDefTarget)componentService);
+    setupOrigin((ServiceDefTarget)adminInfoService);
+    setupOrigin((ServiceDefTarget)tokenAuthService);
+
     Promise.<Config>call(MESSAGES.serverUnavailable(),
         c -> userInfoService.getSystemConfig(sessionId, c))
         .then(result -> {
@@ -2344,6 +2353,20 @@ public boolean getDeleteAccountAllowed() {
     return config.getDeleteAccountAllowed();
   }
 
+  public static void setupOrigin(ServiceDefTarget service) {
+    String host = Window.Location.getProtocol() + "//" + Window.Location.getHost();
+    String oldUrl = service.getServiceEntryPoint();
+    if (oldUrl.startsWith(GWT.getModuleBaseURL())) {
+      String newUrl = host + "/" + GWT.getModuleName() + "/" + oldUrl.substring(GWT.getModuleBaseURL().length());
+      service.setServiceEntryPoint(newUrl);
+    }
+  }
+
+  public static String getModuleBaseURL() {
+      return Window.Location.getProtocol() + "//" + Window.Location.getHost() +
+        "/" + GWT.getModuleName() + "/";
+  }
+
   /**
    * setRendezvousServer
    *

diff --git a/.../appengine/src/com/google/appinventor/client/editor/simple/components/MockFirebaseDB.java b/.../appengine/src/com/google/appinventor/client/editor/simple/components/MockFirebaseDB.java
@@ -17,6 +17,7 @@
 
 import com.google.gwt.core.client.GWT;
 import com.google.gwt.user.client.rpc.AsyncCallback;
+import com.google.gwt.user.client.rpc.ServiceDefTarget;
 import com.google.gwt.user.client.ui.Image;
 import com.google.gwt.user.client.ui.Widget;
 import java.util.logging.Logger;
@@ -54,6 +55,7 @@ public class MockFirebaseDB extends MockNonVisibleComponent {
    */
   public MockFirebaseDB(SimpleEditor editor, String type, Image iconImage) {
     super(editor, type, iconImage);
+    Ode.setupOrigin((ServiceDefTarget)AUTH_SVC);
   }
 
   /**

diff --git a/...or/appengine/src/com/google/appinventor/client/explorer/youngandroid/ProjectListItem.java b/...or/appengine/src/com/google/appinventor/client/explorer/youngandroid/ProjectListItem.java
@@ -48,7 +48,7 @@ public ProjectListItem(Project project) {
     initWidget(UI_BINDER.createAndBindUi(this));
     this.getElement().setAttribute("data-exporturl",
         "application/octet-stream:" + project.getProjectName() + ".aia:"
-            + GWT.getModuleBaseURL() + ServerLayout.DOWNLOAD_SERVLET_BASE
+            + Ode.getModuleBaseURL() + ServerLayout.DOWNLOAD_SERVLET_BASE
             + ServerLayout.DOWNLOAD_PROJECT_SOURCE + "/" + project.getProjectId());
     configureDraggable(this.getElement());
     DateTimeFormat dateTimeFormat = DateTimeFormat.getFormat(DATE_TIME_MEDIUM);

diff --git a/appinventor/appengine/src/com/google/appinventor/client/utils/Downloader.java b/appinventor/appengine/src/com/google/appinventor/client/utils/Downloader.java
@@ -7,6 +7,7 @@
 package com.google.appinventor.client.utils;
 
 import com.google.appinventor.client.ErrorReporter;
+import com.google.appinventor.client.Ode;
 import com.google.gwt.core.client.GWT;
 import com.google.gwt.user.client.Event;
 import com.google.gwt.user.client.ui.Frame;
@@ -45,7 +46,7 @@ private Downloader() {
    */
   public final void download(String path) {
     ErrorReporter.hide();
-    setUrl(GWT.getModuleBaseURL() + path);
+    setUrl(Ode.getModuleBaseURL() + path);
   }
 
   /**

diff --git a/appinventor/appengine/src/com/google/appinventor/client/wizards/ComponentImportWizard.java b/appinventor/appengine/src/com/google/appinventor/client/wizards/ComponentImportWizard.java
@@ -152,7 +152,7 @@ public void execute() {
             return;
           }
 
-          String url = GWT.getModuleBaseURL() +
+          String url = Ode.getModuleBaseURL() +
             ServerLayout.UPLOAD_SERVLET + "/" +
             ServerLayout.UPLOAD_COMPONENT + "/" +
             trimLeadingPath(fileUpload.getFilename());

diff --git a/appinventor/appengine/src/com/google/appinventor/client/wizards/ComponentUploadWizard.java b/appinventor/appengine/src/com/google/appinventor/client/wizards/ComponentUploadWizard.java
@@ -8,6 +8,7 @@
 
 import com.google.appinventor.client.ErrorReporter;
 import static com.google.appinventor.client.Ode.MESSAGES;
+import com.google.appinventor.client.Ode;
 import com.google.appinventor.client.OdeAsyncCallback;
 import com.google.appinventor.client.utils.Uploader;
 import com.google.appinventor.shared.rpc.ServerLayout;
@@ -47,7 +48,7 @@ public void execute() {
           return;
         }
 
-        String url = GWT.getModuleBaseURL() +
+        String url = Ode.getModuleBaseURL() +
           ServerLayout.UPLOAD_SERVLET + "/" +
           ServerLayout.UPLOAD_COMPONENT + "/" +
           trimLeadingPath(uploadWiget.getFilename());

diff --git a/appinventor/appengine/src/com/google/appinventor/client/wizards/FileUploadWizard.java b/appinventor/appengine/src/com/google/appinventor/client/wizards/FileUploadWizard.java
@@ -165,9 +165,9 @@ public void onSuccess(Long date) {
 
       // Use the folderNode's project id and file id in the upload URL so that the file is
       // uploaded into that project and that folder in our back-end storage.
-      String uploadUrl = GWT.getModuleBaseURL() + ServerLayout.UPLOAD_SERVLET + "/" +
-                             ServerLayout.UPLOAD_FILE + "/" + folderNode.getProjectId() + "/" +
-                             folderNode.getFileId() + "/" + filename;
+      String uploadUrl = Ode.getModuleBaseURL() + ServerLayout.UPLOAD_SERVLET + "/" +
+        ServerLayout.UPLOAD_FILE + "/" + folderNode.getProjectId() + "/" +
+        folderNode.getFileId() + "/" + filename;
       Uploader.getInstance().upload(upload, uploadUrl,
           new OdeAsyncCallback<UploadResponse>(MESSAGES.fileUploadError()) {
             @Override

diff --git a/appinventor/appengine/src/com/google/appinventor/client/wizards/KeystoreUploadWizard.java b/appinventor/appengine/src/com/google/appinventor/client/wizards/KeystoreUploadWizard.java
@@ -50,7 +50,7 @@ public KeystoreUploadWizard(final Command callbackAfterUpload) {
       public void execute() {
         String filename = upload.getFilename();
         if (filename.endsWith(KEYSTORE_EXTENSION)) {
-          String uploadUrl = GWT.getModuleBaseURL() + ServerLayout.UPLOAD_SERVLET + "/" +
+          String uploadUrl = Ode.getModuleBaseURL() + ServerLayout.UPLOAD_SERVLET + "/" +
               ServerLayout.UPLOAD_USERFILE + "/" + StorageUtil.ANDROID_KEYSTORE_FILENAME;
           Uploader.getInstance().upload(upload, uploadUrl,
               new OdeAsyncCallback<UploadResponse>(

diff --git a/appinventor/appengine/src/com/google/appinventor/client/wizards/ProjectUploadWizard.java b/appinventor/appengine/src/com/google/appinventor/client/wizards/ProjectUploadWizard.java
@@ -81,7 +81,7 @@ public void getNewName(String name) {
   }
 
   private void upload(FileUpload upload, String filename) {
-    String uploadUrl = GWT.getModuleBaseURL() + ServerLayout.UPLOAD_SERVLET + "/"
+    String uploadUrl = Ode.getModuleBaseURL() + ServerLayout.UPLOAD_SERVLET + "/"
         + ServerLayout.UPLOAD_PROJECT + "/" + filename;
     Uploader.getInstance().upload(upload, uploadUrl,
         new OdeAsyncCallback<UploadResponse>(

diff --git a/appinventor/appengine/src/com/google/appinventor/server/OdeRemoteServiceServlet.java b/appinventor/appengine/src/com/google/appinventor/server/OdeRemoteServiceServlet.java
@@ -40,7 +40,42 @@
  * servlets.
  *
  */
-public abstract class OdeRemoteServiceServlet extends RemoteServiceServlet {
 
+import com.google.gwt.user.server.rpc.SerializationPolicy;
+
+import java.net.MalformedURLException;
+import java.net.URL;
+
+import javax.servlet.http.HttpServletRequest;
+import javax.servlet.http.HttpServletResponse;
+
+public abstract class OdeRemoteServiceServlet extends RemoteServiceServlet {
+  public final static String MODULE_ALIAS = "ode";
   protected final UserInfoProvider userInfoProvider = LocalUser.getInstance();
+
+  @Override
+  protected SerializationPolicy doGetSerializationPolicy(final HttpServletRequest request,
+    final String moduleBaseURL, final String strongName) {
+
+    // true client side relative location is the app name
+    String newModuleBaseURL = moduleBaseURL;
+    try {
+      URL url = new URL(moduleBaseURL);
+
+      StringBuilder builder = new StringBuilder();
+      builder.append(url.getProtocol());
+      builder.append("://");
+      builder.append(url.getHost());
+      builder.append("/");
+      builder.append(MODULE_ALIAS);
+      builder.append("/");
+      newModuleBaseURL = builder.toString();
+
+    } catch (MalformedURLException ex) {
+      // we have no affect
+    }
+
+    return super.doGetSerializationPolicy(request, newModuleBaseURL, strongName);
+  }
+
 }
diff --git a/appinventor/appengine/src/com/google/appinventor/shared/storage/StorageUtil.java b/appinventor/appengine/src/com/google/appinventor/shared/storage/StorageUtil.java
@@ -8,6 +8,7 @@
 
 import com.google.appinventor.shared.rpc.ServerLayout;
 import com.google.gwt.core.client.GWT;
+import com.google.gwt.user.client.Window;
 
 /**
  * Constants and utility methods related to storage.
@@ -251,7 +252,11 @@ public static boolean isFontFile(String filePath)  {
    * Returns the URL for the given project file.
    */
   public static String getFileUrl(long projectId, String fileId) {
-    return GWT.getModuleBaseURL() + getFilePath(projectId, fileId);
+    // Note: Cannot call ode.GetModuleBaseURL() here because it is shared
+    // and Ode is not in scope on the server side.
+    String host = Window.Location.getProtocol() + "//" + Window.Location.getHost();
+    return (host + "/" + GWT.getModuleName() + "/" + getFilePath(projectId, fileId));
+//    return GWT.getModuleBaseURL() + getFilePath(projectId, fileId);
   }
 
   /**

diff --git a/appinventor/appengine/war/WEB-INF/appengine-web.xml b/appinventor/appengine/war/WEB-INF/appengine-web.xml
@@ -178,5 +178,13 @@
     <property name="http.keepAlive" value="true" />
     <property name="http.maxConnections" value="5" />
 
+    <!-- Where to load ode from -->
+
+    <!-- Set this to the base URL of where ode is located. Leaving it blank -->
+    <!-- will load ode from the same host as the server. Note: base URL must be -->
+    <!-- a top level url. Aka https://example.com/ *not* -->
+    <!-- https://example.com/something. -->
+    <property name="ode.base" value="" />
+
   </system-properties>
 </appengine-web-app>
diff --git a/appinventor/appengine/war/index.jsp b/appinventor/appengine/war/index.jsp
@@ -1,4 +1,5 @@
 <%@page import="com.google.appinventor.server.Server,com.google.appinventor.common.version.AppInventorFeatures" %>
+<%@page import="com.google.appinventor.server.flags.Flag" %>
 <%
    if (request.getScheme().equals("http") && Server.isProductionServer()
        && AppInventorFeatures.enableHttpRedirect()) {
@@ -16,6 +17,7 @@
    if (AppInventorFeatures.enableHttpRedirect()) {
        response.setHeader("Strict-Transport-Security", "max-age=3600");
    }
+   final String odeBase = Flag.createFlag("ode.base", "").get();
 %>
 <!-- Copyright 2007-2009 Google Inc. All Rights Reserved. -->
 <!-- Copyright 2011-2020 Massachusetts Institute of Technology. All Rights Reserved. -->
@@ -63,7 +65,7 @@
       </ul>
     </div>
     <script type="text/javascript" src="static/closure-library/closure/goog/base.js"></script>
-    <script type="text/javascript" src="ode/ode.nocache.js"></script>
+    <script type="text/javascript" src="<%= odeBase %>ode/ode.nocache.js"></script>
     <script src="static/leaflet/leaflet.js"></script>
     <script src="static/leaflet/leaflet.toolbar.js"></script>
     <script src="static/leaflet/leaflet-vector-markers.min.js"></script>