Skip to content

Commit 5e6716f

Browse files
committed
document it better
1 parent 3ce6747 commit 5e6716f

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

source/request_response.c

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1036,16 +1036,17 @@ struct aws_http_message *aws_http2_message_new_from_http1(
10361036
struct aws_byte_cursor lower_name_cursor = aws_byte_cursor_from_buf(&lower_name_buf);
10371037
enum aws_http_header_name name_enum = aws_http_lowercase_str_to_header_name(lower_name_cursor);
10381038
switch (name_enum) {
1039+
/**
1040+
* An intermediary transforming an HTTP/1.x message to HTTP/2 MUST remove connection-specific header
1041+
* fields as discussed in Section 7.6.1 of [HTTP]. (RFC=9113 8.2.2)
1042+
*/
10391043
case AWS_HTTP_HEADER_CONNECTION:
10401044
case AWS_HTTP_HEADER_TRANSFER_ENCODING:
10411045
case AWS_HTTP_HEADER_UPGRADE:
10421046
case AWS_HTTP_HEADER_KEEP_ALIVE:
10431047
case AWS_HTTP_HEADER_PROXY_CONNECTION:
1048+
/* Host has been converted to :authority pseudo header, skip it as well. */
10441049
case AWS_HTTP_HEADER_HOST:
1045-
/**
1046-
* An intermediary transforming an HTTP/1.x message to HTTP/2 MUST remove connection-specific header
1047-
* fields as discussed in Section 7.6.1 of [HTTP]. (RFC=9113 8.2.2)
1048-
*/
10491050
AWS_LOGF_TRACE(
10501051
AWS_LS_HTTP_GENERAL,
10511052
"Skip connection-specific headers - \"%.*s\" ",

0 commit comments

Comments
 (0)