arturobernalg commented on PR #635: URL: https://github.com/apache/httpcomponents-core/pull/635#issuecomment-3950372563
> @arturobernalg The PRIORITY mechanism in HEADERS frame has been deprecated https://www.rfc-editor.org/rfc/rfc9113.html#PriorityHere. Why do we need this check? Deprecated priority is about semantics, not framing. If the peer sets HEADERS PRIORITY, the 5-octet fields are still mandatory; otherwise it’s a malformed frame → FRAME_SIZE_ERROR. We must also consume/skip those 5 bytes even if we ignore prioritization, or we’d mis-parse the header block fragment. Also, HEADERS PADDED doesn’t seem to be handled on this path; I’ll open a separate PR to implement PADDED parsing / padding stripping. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
