-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Resource table for plugins #3453
Comments
Yeah, that was done intentionally to not expose anything besides control buffer and zero copy buffer to ops in plugins. It definitely makes sense to give access to |
Probably, unless we add a new context.register_stateful_op function (next to context.register_op) with a different op signature. These types of ops would then have access to the resource table and any other state that is exposed to plugins. That way this can be added without breaking compat (if this happens after 1.0). |
- Removes unnecessary RwLock and Rc around the op registry table - Preparation to move resource_table to deno_core::Isolate. - Towards denoland#3453, denoland#4222
- Removes unnecessary RwLock and Rc around the op registry table - Preparation to move resource_table to deno_core::Isolate. - Towards denoland#3453, denoland#4222
I think plugins should be able to use the resource table to some extent. If this is not the case every plugin will need to build their own resource management system, with extra ops to list and close resources. If plugins can just use the builtin resource table, they can just use Deno.close and Deno.resources to close and list resources.
The text was updated successfully, but these errors were encountered: