Skip to content

policy: cleanup to allow snapshotting #42191

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.

Sign up for GitHub

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

Closed
bmeck wants to merge 9 commits into from
Closed

policy: cleanup to allow snapshotting #42191

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
521f044
policy: cleanup to allow snapshotting
bmeck Mar 2, 2022
28c153a
fixup: lint
bmeck Mar 2, 2022
19e3aa4
fixup: cleanup and rebase
bmeck Mar 18, 2022
2e47f73
fixup: test
bmeck Mar 18, 2022
eb0893a
fixup: clarifying doc on policy.setup(null)
bmeck Mar 18, 2022
ba32054
fixup: lint
bmeck Mar 18, 2022
c85b8e5
fixup: lint
bmeck Mar 18, 2022
1b2d5bb
fixup: review nit
bmeck Mar 22, 2022
d695c78
fixup: CI fix for non-crypto build
bmeck Mar 22, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fixup: clarifying doc on policy.setup(null)
  • Loading branch information
@bmeck
bmeck committed Mar 18, 2022
commit eb0893af5b99593c1efa62040a34a72631eb996f
7 changes: 7 additions & 0 deletions lib/internal/bootstrap/pre_execution.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -482,8 +482,15 @@ function initializePolicy() {
}
policy.setup(src, manifestURL.href);
} else {
// this disables the policy TDZ checks so code can just check
// if (!policy.manifest)
// it is important to keep the policy in TDZ until the user supplied
// options are determined at startup
policy.setup(null);
Comment thread
bmeck marked this conversation as resolved.
}
// ensure that policy no longer has a TDZ by checking
// that we can access .manifest policy.manifest
policy.manifest;
}

function initializeWASI() {
Expand Down