-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
feat(runtime/permissions): prompt fallback #9376
Conversation
4b4c06f
to
0849cdc
Compare
# Conflicts: # runtime/ops/timers.rs
# Conflicts: # runtime/ops/os.rs # runtime/ops/process.rs # runtime/ops/worker_host.rs
# Conflicts: # runtime/permissions.rs
# Conflicts: # runtime/permissions.rs
# Conflicts: # op_crates/websocket/lib.rs # runtime/ops/fs.rs # runtime/ops/net.rs # runtime/ops/os.rs # runtime/ops/process.rs
As far as I tested with 56a7ee6 this looks good to me 👍 It might be better to add unit test of prompt-enabled permissions using |
# Conflicts: # op_crates/fetch/lib.rs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you @crowlKats for seeing this through!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the late review!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
# Conflicts: # .github/workflows/ci.yml
Thank you, Crowl. Nice feature to have! |
This commit adds
--prompt
flag that changes behavior of permissionchecking to displaying interactive prompt instead of failing on missing
permissions.
Based on #9367
Closes #3811