Skip to content

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

Merged
istio-testing merged 1 commit intoistio:release-1.4from
howardjohn:14/cp-24
May 12, 2020
Merged

1.4 - Do not call into the VM unless the VM Context has been created.…#211
istio-testing merged 1 commit intoistio:release-1.4from
howardjohn:14/cp-24

Conversation

@howardjohn
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:]

…envoyproxy#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]>
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