Improve error message when sandbox argument to sandbox_invoke is missing

Open deian opened this issue 6 years ago • 3 comments

Dec 28 '19 07:12 deian

Hi @deian, are we talking about the invoke_sandbox_function ? Also just to confirm, since we don't actually pass the sandbox as an argument, are we looking for an error when the sandbox is missing, or for the function name that is to be invoked?

Apr 09 '20 16:04 rhl-bthr

Close, but not exactly. We're talking about sandbox_invoke, which expects a sandbox as a the first argument.

Though @shravanrn is the plan to move everything to invoke_sandbox_function?

Apr 10 '20 20:04 deian

Actually, we intend to support both going forward. But yes, the error message is not very helpful when the argument is missing.

Apr 10 '20 20:04 shravanrn