Skip to content

Commit 3a7c1bc

Browse files
committed
Feat: Add 'Plugin Crash' to messages logged when plugins crash to improve error observability.
Signed-off-by: Rachel Green <rachgreen@google.com>
1 parent 068b288 commit 3a7c1bc

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

src/v8/v8.cc

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ class V8 : public WasmVm {
108108
private:
109109
wasm::own<wasm::Trap> trap(std::string message);
110110

111-
std::string getFailMessage(std::string_view function_name, wasm::own<wasm::Trap> trap);
111+
std::string getPluginFailMessage(std::string_view function_name, wasm::own<wasm::Trap> trap);
112112

113113
template <typename... Args>
114114
void registerHostFunctionImpl(std::string_view module_name, std::string_view function_name,
@@ -650,7 +650,8 @@ void V8::getModuleFunctionImpl(std::string_view function_name,
650650
}
651651

652652
if (trap) {
653-
fail(FailState::RuntimeError, getFailMessage(std::string(function_name), std::move(trap)));
653+
fail(FailState::RuntimeError,
654+
getPluginFailMessage(std::string(function_name), std::move(trap)));
654655
return;
655656
}
656657
if (log) {
@@ -702,7 +703,8 @@ void V8::getModuleFunctionImpl(std::string_view function_name,
702703
}
703704

704705
if (trap) {
705-
fail(FailState::RuntimeError, getFailMessage(std::string(function_name), std::move(trap)));
706+
fail(FailState::RuntimeError,
707+
getPluginFailMessage(std::string(function_name), std::move(trap)));
706708
return R{};
707709
}
708710
R rvalue = results[0].get<typename ConvertWordTypeToUint32<R>::type>();
@@ -722,8 +724,8 @@ void V8::terminate() {
722724

723725
void V8::warm() { initStore(); }
724726

725-
std::string V8::getFailMessage(std::string_view function_name, wasm::own<wasm::Trap> trap) {
726-
auto message = "Function: " + std::string(function_name) + " failed: ";
727+
std::string V8::getPluginFailMessage(std::string_view function_name, wasm::own<wasm::Trap> trap) {
728+
auto message = "Plugin Crash: Function: " + std::string(function_name) + " failed: ";
727729
message += std::string(trap->message().get(), trap->message().size());
728730

729731
if (function_names_index_.empty()) {

0 commit comments

Comments
 (0)