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

E and harden #679

Open
dckc opened this issue Jun 8, 2022 · 5 comments
Open

E and harden #679

dckc opened this issue Jun 8, 2022 · 5 comments

Comments

@dckc
Copy link
Member

dckc commented Jun 8, 2022

It’s E(target)(…args) that hardens args. -- @michaelfig

@dckc
Copy link
Member Author

dckc commented Aug 2, 2022

The general rule is:

all objects made by literal expressions (object literals, array literals, the many forms of function literals) must be tamper-proofed with harden before they can be aliased or escape from their static context of origin -- https://github.com/endojs/Jessie#must-freeze-api-surface-before-use

As a practical concession, since any call to E(target).method(...args) may be remote, we automatically harden the args, since @endo/marshal requires it inputs to be hardened.

@Tyrosine22
Copy link
Contributor

This seems to be correctly documented now.

@dckc
Copy link
Member Author

dckc commented Jan 10, 2023

I don't see it in the eventual send section. Where is it?

@dckc dckc removed the wontfix label Jan 23, 2023
0xpatrickdev added a commit to 0xpatrickdev/agoric-documentation that referenced this issue Jul 11, 2023
@dckc
Copy link
Member Author

dckc commented Feb 3, 2024

41469e7 looks like a keeper

@dckc
Copy link
Member Author

dckc commented Jan 22, 2025

@yribero ran into this today... got "not writeable"

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

No branches or pull requests

4 participants