Skip to content

fix(updater): fallback to passive mode & fix installerArgs deserialzation#1051

Merged
amrbashir merged 9 commits into
tauri-apps:v2from
Legend-Master:updater-windows-field-fix
Mar 11, 2024
Merged

fix(updater): fallback to passive mode & fix installerArgs deserialzation#1051
amrbashir merged 9 commits into
tauri-apps:v2from
Legend-Master:updater-windows-field-fix

Conversation

@Legend-Master
Copy link
Copy Markdown
Contributor

Fix #1015

Also fixes installerArgs not working in tauri config files

- Also fix installerArgs not working in tauri config
@Legend-Master Legend-Master requested a review from a team as a code owner March 8, 2024 02:53
Comment thread plugins/updater/src/config.rs Outdated
Comment thread plugins/updater/src/config.rs Outdated
@Legend-Master Legend-Master requested a review from amrbashir March 8, 2024 06:13
@Legend-Master
Copy link
Copy Markdown
Contributor Author

Not sure why it can't pick up installer_args if windows field is Option<WindowsConfig>, WindowsConfig works though

@Legend-Master
Copy link
Copy Markdown
Contributor Author

Strange, cargo test --bin app-updater on my machine succeed

@Legend-Master
Copy link
Copy Markdown
Contributor Author

Legend-Master commented Mar 8, 2024

Found the problem, this part is still handled from tauri itself, not sure why changing installer_args to be not an Option fixes the problem

https://github.com/tauri-apps/tauri/blob/45384ab3aa44687f25323ebf367058270ec3a4d3/tooling/cli/src/interface/rust.rs#L780-L783

Copy link
Copy Markdown
Member

@amrbashir amrbashir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the later reply, I have decided to revert back to OsString and go with your original solution, deserialize from string first.

Thanks for contribution

@amrbashir amrbashir merged commit 4e37316 into tauri-apps:v2 Mar 11, 2024
@amrbashir amrbashir changed the title fix(updater): windows field is none by default fix(updater): fallback to passive mode & fix installerArgs deserialzation Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants