Skip to content

Conversation

Legend-Master
Copy link
Contributor

@Legend-Master Legend-Master commented Sep 1, 2025

It's not obvious that this permission should be used in this syntax:

{
  "identifier": "fs:scope",
  "allow": [
    "$APPDATA/documents/**/*"
  ],
  "deny": [
    "$APPDATA/documents/secret.txt"
  ]
}

adding in an example for this

@Legend-Master Legend-Master requested a review from a team as a code owner September 1, 2025 15:02
Copy link
Contributor

github-actions bot commented Sep 1, 2025

Package Changes Through b0640c8

There are 4 changes which include geolocation with patch, geolocation-js with patch, updater with minor, updater-js with minor

Planned Package Versions

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

package current next
api-example 2.0.36 2.0.37
api-example-js 2.0.32 2.0.33
geolocation 2.3.0 2.3.1
geolocation-js 2.3.0 2.3.1
updater 2.9.0 2.10.0
updater-js 2.9.0 2.10.0

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

I think we shouldn't use ** to match files in our docs as people confuse that with **/* all the time (or was the latter what you meant to use?)

@Legend-Master
Copy link
Contributor Author

Legend-Master commented Sep 1, 2025

I think we shouldn't use ** to match files in our docs as people confuse that with **/* all the time (or was the latter what you meant to use?)

Nice catch, guess I'm one of them 😂, I meant to use **/*, to be honest though, ** really feels like it could match everything recursively
Also the glob documentation sucked, I wish I learned it through https://docs.rs/glob/latest/glob/struct.Pattern.html

@Legend-Master Legend-Master merged commit 92f98a9 into tauri-apps:v2 Oct 1, 2025
7 checks passed
@Legend-Master Legend-Master deleted the fs-scope-docs-example branch October 1, 2025 09:24
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