Skip to content

1.4 - Do not call into the VM unless the VM Context has been created.…#212

Closed
brian-avery wants to merge 1 commit intoistio:release-1.4from
brian-avery:149Update
Closed

1.4 - Do not call into the VM unless the VM Context has been created.…#212
brian-avery wants to merge 1 commit intoistio:release-1.4from
brian-avery:149Update

Conversation

@brian-avery
Copy link
Copy Markdown
Member

… (#24)

  • Ensure that the in VM Context is created before onDone is called.

Signed-off-by: John Plevyak [email protected]

  • Update as per offline discussion.

Signed-off-by: John Plevyak [email protected]

  • Set in_vm_context_created_ in onNetworkNewConnection.

Signed-off-by: John Plevyak [email protected]

  • Add guards to other network calls.

Signed-off-by: John Plevyak [email protected]

  • Fix common/wasm tests.

Signed-off-by: John Plevyak [email protected]

  • Patch tests.

Signed-off-by: John Plevyak [email protected]

  • Remove unecessary file from cherry-pick.

Signed-off-by: John Plevyak [email protected]

For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md

Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

…#24)

* Ensure that the in VM Context is created before onDone is called.

Signed-off-by: John Plevyak <[email protected]>

* Update as per offline discussion.

Signed-off-by: John Plevyak <[email protected]>

* Set in_vm_context_created_ in onNetworkNewConnection.

Signed-off-by: John Plevyak <[email protected]>

* Add guards to other network calls.

Signed-off-by: John Plevyak <[email protected]>

* Fix common/wasm tests.

Signed-off-by: John Plevyak <[email protected]>

* Patch tests.

Signed-off-by: John Plevyak <[email protected]>

* Remove unecessary file from cherry-pick.

Signed-off-by: John Plevyak <[email protected]>
@PiotrSikora
Copy link
Copy Markdown

Duplicate of #211.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants