-
Notifications
You must be signed in to change notification settings - Fork 136
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
Spawning operating system processes #149
Comments
I personally think having both a whitelist and blacklist of input filtering, with the blacklist taking precedence in case of collisions, would go a long way towards having an initial security implementation for this. Hopefully others will comment here as well. |
I think this would be useful and got here because I was thinking about whether I could build a distributed It shouldn't be an on-by-default capability and the docs should warn users and encourage them to compile their utilities to WASM and run them inside lunatic instead if possible, but that's not always practical (closed source binaries, maintainers not interested in WASM support, reliance on APIs that don't exist in WASI yet, better native performance, etc). |
I am a newbie to Lunatic, WebAssembly and Rust, so please bare with my queries. I am looking into the issue and see if i can implement it.
So let's give it capabilities to access files in the requisite directories:
Do we need to consider this when we allow which all commands can be executed and whether the restriction is applicable to those commands?
@bkolobara gave me some guidance (async, streaming etc) in the discord channel. I will have better clarity (and hopefully better queries) once i complete reading of those links. |
Sometimes it's useful to just offload the workload to some command line tool. Under the hood, it would just use Rust's process::Command.
We would also need an additional configuration option to add an allow-list for specific processes. By default it would be empty.
It would also be useful to capture the status and output.
The text was updated successfully, but these errors were encountered: