http2: do not crash on mismatched ping buffer length

[Renegade334]

const session = require('http2').connect('http://localhost/')
session.ping(new Uint32Array(8), console.debug)

yields

  #  node[1898056]: static void node::http2::Http2Session::Ping(const v8::FunctionCallbackInfo<v8::Value>&) at ../src/node_http2.cc:3227
  #  Assertion failed: (payload.length()) == (8)

IOT instruction (core dumped)

[nodejs-github-bot]

Review requested:

• @nodejs/http2
• @nodejs/net

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.52%. Comparing base (b757a8f) to head (b0a2c8c).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #60135      +/-   ##
==========================================
- Coverage   88.53%   88.52%   -0.02%     
==========================================
  Files         703      703              
  Lines      207804   207804              
  Branches    40005    40006       +1     
==========================================
- Hits       183989   183952      -37     
- Misses      15817    15866      +49     
+ Partials     7998     7986      -12     

Files with missing lines	Coverage Δ
lib/internal/http2/core.js	95.20% <100.00%> (ø)

... and 30 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/69586/

[mcollina]

lgtm

[nodejs-github-bot]

Landed in 535efea