Skip to content

Conversation

@catalinsh
Copy link
Contributor

According to the NSIS docs RequestExecutionLevel highest will request the highest execution level available for the current user.

This makes it so that, if the user launching the installer is not an Administrator, the installer will not ask to be elevated and the installation will fail when reaching a step that requires admin priviledges.

Switching to RequestExecutionLevel admin ensures the installer always asks for elevation.

@catalinsh catalinsh requested a review from a team as a code owner July 8, 2025 20:40
@github-project-automation github-project-automation bot moved this to 📬Proposal in Roadmap Jul 8, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jul 8, 2025

Package Changes Through 44f10e9

There are 5 changes which include @tauri-apps/cli with patch, tauri-cli with patch, @tauri-apps/api with patch, tauri-bundler with patch, tauri-utils with patch

Planned Package Versions

The following package releases are the planned based on the context of changes in this pull request.

package current next
@tauri-apps/api 2.6.0 2.6.1
tauri-utils 2.5.0 2.5.1
tauri-bundler 2.5.0 2.5.1
tauri-runtime 2.7.0 2.7.1
tauri-runtime-wry 2.7.1 2.7.2
tauri-codegen 2.3.0 2.3.1
tauri-macros 2.3.1 2.3.2
tauri-plugin 2.3.0 2.3.1
tauri-build 2.3.0 2.3.1
tauri 2.6.2 2.6.3
@tauri-apps/cli 2.6.2 2.6.3
tauri-cli 2.6.2 2.6.3

Add another change file through the GitHub UI by following this link.


Read about change files or the docs at github.com/jbolda/covector

@FabianLars
Copy link
Member

thanks for the pr :)

for the reviewers someone else(?) confirmed that this works as well: https://discord.com/channels/616186924390023171/1381811624624652331

Copy link
Contributor

@Legend-Master Legend-Master left a comment

Choose a reason for hiding this comment

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

Makes sense to me, nice catch

They correspond to requireAdministrator and highestAvailable in case anyone also wants to take a look

https://learn.microsoft.com/en-us/windows/win32/sbscs/application-manifests#trustinfo

@FabianLars FabianLars merged commit f94af90 into tauri-apps:dev Jul 9, 2025
14 checks passed
@github-project-automation github-project-automation bot moved this from 📬Proposal to 🔎 In audit in Roadmap Jul 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🔎 In audit

Development

Successfully merging this pull request may close these issues.

3 participants