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

Restrict and document actual required permissions #374

Open
jsejcksn opened this issue Jul 2, 2022 · 1 comment
Open

Restrict and document actual required permissions #374

jsejcksn opened this issue Jul 2, 2022 · 1 comment
Labels
feat New feature or request

Comments

@jsejcksn
Copy link

jsejcksn commented Jul 2, 2022

Documentation and code in this repo use deno run -A (--allow-all). This:

  • disable's Deno's runtime security sandbox
  • is opaque and counter to a zero-trust model (upon which that sandbox is based)

I understand that some tasks in Fresh are completed using CLI calls to deno, and therefore currently require --allow-run (which is essentially the same as --allow-all because it allows for privilege escalation), but I'm not sure if they have to if some changes are made. The usages I see in a precursory search are:

Apart from those, the other required permissions should be clearly documented so that users can run fresh with minimal and predefined permission grants instead of disabling the security sandbox entirely.

@lucacasonato lucacasonato added the feat New feature or request label Jan 12, 2023
@lucacasonato
Copy link
Member

I'd accept a PR from someone willing to address this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants