Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New profile: obsidian #6314

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Prev Previous commit
Next Next commit
Update etc/profile-m-z/obsidian.profile
  • Loading branch information
glitsj16 committed Apr 25, 2024
commit d90938789b9ea1cbf841b499d4b1d266cd06e0cf
2 changes: 1 addition & 1 deletion etc/profile-m-z/obsidian.profile
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ noroot
protocol unix,inet,inet6
#net none
Copy link
Collaborator

Choose a reason for hiding this comment

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

What does it do network-wise?

Just download and update plugins?

I'd leave a comment to clarify:

Suggested change
#net none
#net none # networking is needed to download/update plugins

Copy link
Author

Choose a reason for hiding this comment

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

Check for obsidian updates, check for plugin updates, review and install plugins in the built in plugin manager, and network features for plugins that use them. In other words, if the user allows networking, it also allows network access to any arbitrary plugin.


private-bin bash,basename,cat,cut,electron28,gawk,grep,obsidian,realpath,tr
private-bin basename,bash,cat,cut,electron,electron[0-9],electron[0-9][0-9],gawk,grep,obsidian,realpath,tr
private-etc @network,@tls-ca,@x11,gnutls,libva.conf

# Redirect
Expand Down
Loading