react-native-update 7.3.2 → 7.4.0-beta.0

package/android/src/main/java/cn/reactnative/modules/update/SafeZipFile.java

@@ -16,40 +16,6 @@ public class SafeZipFile extends ZipFile {
         super(file);
     }
 
-    @Override
-    public Enumeration<? extends ZipEntry> entries() {
-        return new SafeZipEntryIterator(super.entries());
-    }
-
-    private static class SafeZipEntryIterator implements Enumeration<ZipEntry> {
-
-        final private Enumeration<? extends ZipEntry> delegate;
-
-        private SafeZipEntryIterator(Enumeration<? extends ZipEntry> delegate) {
-            this.delegate = delegate;
-        }
-
-        @Override
-        public boolean hasMoreElements() {
-            return delegate.hasMoreElements();
-        }
-
-        @Override
-        public ZipEntry nextElement() {
-            ZipEntry entry = delegate.nextElement();
-            if (null != entry) {
-                String name = entry.getName();
-                /**
-                 * avoid ZipperDown
-                 */
-                if (null != name && (name.contains("../") || name.contains("..\\"))) {
-                    throw new SecurityException("illegal entry: " + entry.getName());
-                }
-            }
-            return entry;
-        }
-    }
-
     public void unzipToFile(ZipEntry entry, File output) throws IOException {
         InputStream inputStream = null;
         try {
@@ -63,6 +29,11 @@ public class SafeZipFile extends ZipFile {
     }
 
     private void writeOutInputStream(File file, InputStream inputStream) throws IOException {
+        // https://support.google.com/faqs/answer/9294009
+        String canonicalPath = file.getCanonicalPath();
+        if (!canonicalPath.startsWith(UpdateContext.getRootDir())) {
+            throw new SecurityException("illegal entry: " + file.getName());
+        }
         BufferedOutputStream output = null;
         try {
             output = new BufferedOutputStream(

package/android/src/main/java/cn/reactnative/modules/update/UpdateContext.java

@@ -19,13 +19,14 @@ import java.io.File;
 
 public class UpdateContext {
     private Context context;
-    private File rootDir;
+    private static File rootDir;
     private Executor executor;
 
     public static boolean DEBUG = false;
     private static ReactInstanceManager mReactInstanceManager;
     private static boolean isUsingBundleUrl = false;
 
+
     public UpdateContext(Context context) {
         this.context = context;
         this.executor = Executors.newSingleThreadExecutor();
@@ -49,7 +50,7 @@ public class UpdateContext {
         }
     }
 
-    public String getRootDir() {
+    public static String getRootDir() {
         return rootDir.toString();
     }
 

package/lib/main.js

@@ -138,7 +138,7 @@ export async function checkUpdate(APPKEY, isRetry) {
     });
   } catch (e) {
     if (isRetry) {
-      throw new Error('Could not connect to pushy server');
+      throw new Error('无法连接更新服务器，请检查网络连接后重试');
     }
     await tryBackupEndpoints(APPKEY);
     return checkUpdate(APPKEY, true);
@@ -227,7 +227,7 @@ export async function downloadUpdate(options, eventListeners) {
       } catch (e) {
         progressHandler && progressHandler.remove();
         report(options.hash, 'error');
-        return;
+        throw e;
       }
     }
   } else if (options.pdiffUrl) {
@@ -240,7 +240,7 @@ export async function downloadUpdate(options, eventListeners) {
     } catch (e) {
       progressHandler && progressHandler.remove();
       report(options.hash, 'error');
-      return;
+      throw e;
     }
   }
   setLocalHashInfo(options.hash, {

package/lib/simpleUpdate.js

@@ -36,7 +36,7 @@ export function simpleUpdate(WrappedComponent) {
           this.checkUpdate();
         }
         componentWillUnmount() {
-          this.stateListener.remove();
+          this.stateListener && this.stateListener.remove();
         }
         doUpdate = async (info) => {
           try {
@@ -44,7 +44,7 @@ export function simpleUpdate(WrappedComponent) {
             if (!hash) {
               return;
             }
-            this.stateListener.remove();
+            this.stateListener && this.stateListener.remove();
             Alert.alert('提示', '下载完毕，是否立即更新?', [
               {
                 text: '以后再说',
@@ -94,7 +94,7 @@ export function simpleUpdate(WrappedComponent) {
                 },
               },
             ]);
-          } else {
+          } else if (info.update) {
             Alert.alert(
               '提示',
               '检查到新的版本' + info.name + ',是否下载?\n' + info.description,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-update",
-  "version": "7.3.2",
+  "version": "7.4.0-beta.0",
   "description": "react-native hot update",
   "main": "lib/index.js",
   "scripts": {
@@ -28,6 +28,6 @@
   },
   "homepage": "https://github.com/reactnativecn/react-native-pushy#readme",
   "dependencies": {
-    "nanoid": "^3.1.28"
+    "nanoid": "^3.1.30"
   }
 }