-
Notifications
You must be signed in to change notification settings - Fork 71
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
fix(ses): Add HandledPromise to the whitelist #416
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nits, and please get a signature from @erights for this expedient.
packages/ses/src/whitelist.js
Outdated
@@ -59,6 +59,7 @@ export const universalPropertyNames = { | |||
Map: 'Map', | |||
Number: 'Number', | |||
Object: 'Object', | |||
HandledPromise: 'HandledPromise', // TODO: Until Promise.delegate. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
More information on the TODO will make it more clear when TODO can be removed.
Please add an entry to NEWS.md. |
72ac6be
to
c571a74
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
ba2a9bb
to
c60b7a6
Compare
Done. |
This is a temporary fix until lockdown gets a more disciplined way of updating the whitelist with vetted shims. In any event, it is a step toward
Promise.delegate
, but that standard hasn't yet been implemented in@agoric/eventual-send/shim
.@kriskowal Please update and release a new patch version of
ses
, so that we can use this in agoric-sdk.