fix(macOS): Tauri V2 Update Permission Denied Error (#2067)

* WIP * Fixed linting * WIP * Fixed linting * use osakit to show actual app name on dialog * sync versions * lint --------- Co-authored-by: Lucas Nogueira <[email protected]>
tauri-apps · Feb 2, 2025 · 5369898 · 5369898
1 parent a7497b0
commit 5369898
Show file tree

Hide file tree

Showing 6 changed files with 171 additions and 52 deletions.
diff --git a/.changes/fix-macos-user-install-update.md b/.changes/fix-macos-user-install-update.md
@@ -0,0 +1,6 @@
+---
+"updater": patch
+"updater-js": patch
+---
+
+Fix update installation on macOS when using an user without admin privileges.
diff --git a/.changes/updater-new-fn.md b/.changes/updater-new-fn.md
@@ -0,0 +1,6 @@
+---
+"updater": minor
+"updater-js": minor
+---
+
+Remove the `UpdaterBuilder::new` function, use `UpdaterExt::updater_builder` instead.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/plugins/updater/Cargo.toml b/plugins/updater/Cargo.toml
@@ -62,6 +62,7 @@ flate2 = { version = "1", optional = true }
 [target."cfg(target_os = \"macos\")".dependencies]
 tar = "0.4"
 flate2 = "1"
+osakit = { version = "0.3", features = ["full"] }
 
 [features]
 default = ["rustls-tls", "zip"]

diff --git a/plugins/updater/src/lib.rs b/plugins/updater/src/lib.rs
@@ -70,20 +70,14 @@ pub trait UpdaterExt<R: Runtime> {
 impl<R: Runtime, T: Manager<R>> UpdaterExt<R> for T {
     fn updater_builder(&self) -> UpdaterBuilder {
         let app = self.app_handle();
-        let package_info = app.package_info();
         let UpdaterState {
             config,
             target,
             version_comparator,
             headers,
         } = self.state::<UpdaterState>().inner();
 
-        let mut builder = UpdaterBuilder::new(
-            package_info.name.clone(),
-            package_info.version.clone(),
-            config.clone(),
-        )
-        .headers(headers.clone());
+        let mut builder = UpdaterBuilder::new(app, config.clone()).headers(headers.clone());
 
         if let Some(target) = target {
             builder = builder.target(target);