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

Improve error message when sandbox argument to sandbox_invoke is missing #18

Open
deian opened this issue Dec 28, 2019 · 3 comments
Open

Comments

@deian
Copy link
Member

deian commented Dec 28, 2019

No description provided.

@rhl-bthr
Copy link

rhl-bthr commented Apr 9, 2020

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?

@deian
Copy link
Member Author

deian commented Apr 10, 2020

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?

@shravanrn
Copy link
Collaborator

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

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

No branches or pull requests

3 participants