nodejs/llhttp

How to handle HPE_PAUSED_H2_UPGRADE on upgrade to http2 ?

Opened this issue · 1 comments

The README mentions how to handle HPE_PAUSED_UPGRADE but not HPE_PAUSED_H2_UPGRADE. What should be done to handle this ?

Hi!
You are totally right. HPE_PAUSED_H2_UPGRADE should be handle exactly as HPE_PAUSED_UPGRADE: the parser will return one of those errors codes accordingly to the headers and then pause. Afterwards it can only be unpaused with llhttp_resume_after_upgrade.

Would you be open to send a PR to fix the documentation?