We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
We currently have some vestigial OCap related code that is dysfunctional and should be removed from the ibc crate.
Further to the excellent points raised by @plafer (https://github.com/informalsystems/ibc-rs/issues/2013#issuecomment-1090965987), and other related discussions (https://github.com/informalsystems/ibc-rs/issues/2013), I think we agree that it doesn't make sense to have an OCap facility in a system with untrusted modules and without strictly enforced memory isolation. We currently use OCaps for two things - binding modules to ports and tracking channel/port ownership, so we need to figure out what exactly must be removed.
Note: this may be considered a divergence from the ICS spec.
Any unused OCap related code is removed from the ibc crate.
The text was updated successfully, but these errors were encountered:
plafer
No branches or pull requests
Summary
We currently have some vestigial OCap related code that is dysfunctional and should be removed from the ibc crate.
Problem Definition
Further to the excellent points raised by @plafer (https://github.com/informalsystems/ibc-rs/issues/2013#issuecomment-1090965987), and other related discussions (https://github.com/informalsystems/ibc-rs/issues/2013), I think we agree that it doesn't make sense to have an OCap facility in a system with untrusted modules and without strictly enforced memory isolation.
We currently use OCaps for two things - binding modules to ports and tracking channel/port ownership, so we need to figure out what exactly must be removed.
Note: this may be considered a divergence from the ICS spec.
Acceptance Criteria
Any unused OCap related code is removed from the ibc crate.
For Admin Use
The text was updated successfully, but these errors were encountered: