Skip to content

src,http: fix uncaughtException miss in http

Rodrigo Muino Tomonari requested to merge github/fork/trevnorris/retro-trycatch-mc into v5.x

Pull Request check-list

  • Does make -j8 test (UNIX) or vcbuild test nosign (Windows) pass with this change (including linting)?
  • Is the commit message formatted according to [CONTRIBUTING.md][0]?
  • If this change fixes a bug (or a performance problem), is a regression test (or a benchmark) included?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Description

If the call to MakeCallback is currently in_makecallback() then return
the empty handle instead of Undefined. This allows the error to
propagate through multiple MakeCallback calls.

R=@misterdjules R=@indutny R=@TheAlphaNerd

Merge request reports