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 jplevyak@gmail.com

  • Update as per offline discussion.

Signed-off-by: John Plevyak jplevyak@gmail.com

  • Set in_vm_context_created_ in onNetworkNewConnection.

Signed-off-by: John Plevyak jplevyak@gmail.com

  • Add guards to other network calls.

Signed-off-by: John Plevyak jplevyak@gmail.com

  • Fix common/wasm tests.

Signed-off-by: John Plevyak jplevyak@gmail.com

  • Patch tests.

Signed-off-by: John Plevyak jplevyak@gmail.com

  • Remove unecessary file from cherry-pick.

Signed-off-by: John Plevyak jplevyak@gmail.com

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 <jplevyak@gmail.com>

* Update as per offline discussion.

Signed-off-by: John Plevyak <jplevyak@gmail.com>

* Set in_vm_context_created_ in onNetworkNewConnection.

Signed-off-by: John Plevyak <jplevyak@gmail.com>

* Add guards to other network calls.

Signed-off-by: John Plevyak <jplevyak@gmail.com>

* Fix common/wasm tests.

Signed-off-by: John Plevyak <jplevyak@gmail.com>

* Patch tests.

Signed-off-by: John Plevyak <jplevyak@gmail.com>

* Remove unecessary file from cherry-pick.

Signed-off-by: John Plevyak <jplevyak@gmail.com>
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