Skip to content

fix: allow double -- in permissions as command can have double __ - #12884

Merged
lucasfernog merged 6 commits into
tauri-apps:devfrom
oscartbeaumont:fix-permission-naming
Mar 14, 2025
Merged

fix: allow double -- in permissions as command can have double __#12884
lucasfernog merged 6 commits into
tauri-apps:devfrom
oscartbeaumont:fix-permission-naming

Conversation

@oscartbeaumont

@oscartbeaumont oscartbeaumont commented Mar 3, 2025

Copy link
Copy Markdown
Member

A command in Tauri can technically be called Something__Else (as it's a valid Rust identifier) which would map to allow-Something--Else in the default.toml permissions file, however the permissions file rejects the double -- so in practice it's impossible to grant the permission to run a command with a __ in it's name.

@oscartbeaumont
oscartbeaumont requested a review from a team as a code owner March 3, 2025 14:42
@github-actions

github-actions Bot commented Mar 3, 2025

Copy link
Copy Markdown
Contributor

Package Changes Through 8e1e7f4

There are 7 changes which include tauri-utils with patch, tauri with minor, @tauri-apps/api with minor, tauri-runtime-wry with minor, tauri-bundler with patch, tauri-cli with patch, @tauri-apps/cli 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.3.0 2.4.0
tauri-utils 2.2.0 2.2.1
tauri-bundler 2.2.4 2.2.5
tauri-runtime 2.4.0 2.4.1
tauri-runtime-wry 2.4.1 2.5.0
tauri-codegen 2.0.5 2.0.6
tauri-macros 2.0.5 2.0.6
tauri-plugin 2.0.5 2.0.6
tauri-build 2.0.6 2.0.7
tauri 2.3.1 2.4.0
@tauri-apps/cli 2.3.1 2.3.2
tauri-cli 2.3.1 2.3.2

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


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

Comment thread .changes/change-pr-12884.md Outdated
@lucasfernog
lucasfernog merged commit 3cc4ad3 into tauri-apps:dev Mar 14, 2025
@oscartbeaumont
oscartbeaumont deleted the fix-permission-naming branch March 14, 2025 15:27
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.

3 participants