Skip to content

Commit

Permalink
Migrate to a new V8 interceptors Api (nodejs#180)
Browse files Browse the repository at this point in the history
The new callback should return v8::Intercepted::kYes/kNo to indicate
whether the operation was intercepted. This replaces the old approach
where the callback had to leave the return value unset or set it to
an empty handle to indicate that the the request wasn't intercepted.

See https://crrev.com/c/5465509 and https://crrev.com/c/5465513.
# Conflicts:
#	src/node_contextify.cc

# Conflicts:
#	src/node_contextify.cc
#	src/node_env_var.cc
  • Loading branch information
victorgomes committed Aug 13, 2024
1 parent 01ebad9 commit 7e16a29
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion src/node_env_var.cc
Original file line number Diff line number Diff line change
Expand Up @@ -386,7 +386,6 @@ static Intercepted EnvSetter(Local<Name> property,
}

env->env_vars()->Set(env->isolate(), key, value_string);

return Intercepted::kYes;
}

Expand Down

0 comments on commit 7e16a29

Please sign in to comment.