michaelh/zephyr
1
0
Fork 0
Code Releases Activity

tests: net: lib: http_server: core: Add tests for frames with padding

Browse source 
Add tests verifying that requests which include frames with padding are
processed correctly.

Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
This commit is contained in:
Robert Lubos 2024-07-17 15:47:26 +02:00 • committed by Anas Nashif
commit 1da42b6626
1 changed files with 80 additions and 21 deletions
Download patch file Download diff file Expand all files Collapse all files

101
tests/net/lib/http_server/core/src/main.c
View file

@ -55,6 +55,14 @@
0x86, 0x62, 0x4f, 0x55, 0x0e, 0x93, 0x13, 0x7a, 0x88, 0x25, 0xb6, 0x50, \ 0x86, 0x62, 0x4f, 0x55, 0x0e, 0x93, 0x13, 0x7a, 0x88, 0x25, 0xb6, 0x50, \
0xc3, 0xcb, 0xbc, 0xb8, 0x3f, 0x53, 0x03, 0x2a, 0x2f, 0x2a, 0x5f, 0x87, \ 0xc3, 0xcb, 0xbc, 0xb8, 0x3f, 0x53, 0x03, 0x2a, 0x2f, 0x2a, 0x5f, 0x87, \
0x49, 0x7c, 0xa5, 0x8a, 0xe8, 0x19, 0xaa 0x49, 0x7c, 0xa5, 0x8a, 0xe8, 0x19, 0xaa
#define TEST_HTTP2_HEADERS_GET_DYNAMIC_STREAM_1_PADDED \
0x00, 0x00, 0x3d, 0x01, 0x0d, 0x00, 0x00, 0x00, TEST_STREAM_ID_1, 0x11,\
0x82, 0x86, 0x41, 0x87, 0x0b, 0xe2, 0x5c, 0x0b, 0x89, 0x70, 0xff, 0x04, \
0x86, 0x62, 0x4f, 0x55, 0x0e, 0x93, 0x13, 0x7a, 0x88, 0x25, 0xb6, 0x50, \
0xc3, 0xcb, 0xbc, 0xb8, 0x3f, 0x53, 0x03, 0x2a, 0x2f, 0x2a, 0x5f, 0x87, \
0x49, 0x7c, 0xa5, 0x8a, 0xe8, 0x19, 0xaa, \
0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, \
0x00, 0x00, 0x00, 0x00, 0x00
#define TEST_HTTP2_HEADERS_POST_DYNAMIC_STREAM_1 \ #define TEST_HTTP2_HEADERS_POST_DYNAMIC_STREAM_1 \
0x00, 0x00, 0x30, 0x01, 0x04, 0x00, 0x00, 0x00, TEST_STREAM_ID_1, \ 0x00, 0x00, 0x30, 0x01, 0x04, 0x00, 0x00, 0x00, TEST_STREAM_ID_1, \
0x83, 0x86, 0x41, 0x87, 0x0b, 0xe2, 0x5c, 0x0b, 0x89, 0x70, 0xff, 0x04, \ 0x83, 0x86, 0x41, 0x87, 0x0b, 0xe2, 0x5c, 0x0b, 0x89, 0x70, 0xff, 0x04, \
@ -65,6 +73,14 @@
0x00, 0x00, 0x11, 0x00, 0x01, 0x00, 0x00, 0x00, TEST_STREAM_ID_1, \ 0x00, 0x00, 0x11, 0x00, 0x01, 0x00, 0x00, 0x00, TEST_STREAM_ID_1, \
0x54, 0x65, 0x73, 0x74, 0x20, 0x64, 0x79, 0x6e, 0x61, 0x6d, 0x69, 0x63, \ 0x54, 0x65, 0x73, 0x74, 0x20, 0x64, 0x79, 0x6e, 0x61, 0x6d, 0x69, 0x63, \
0x20, 0x50, 0x4f, 0x53, 0x54 0x20, 0x50, 0x4f, 0x53, 0x54
#define TEST_HTTP2_DATA_POST_DYNAMIC_STREAM_1_PADDED \
0x00, 0x00, 0x34, 0x00, 0x09, 0x00, 0x00, 0x00, TEST_STREAM_ID_1, 0x22, \
0x54, 0x65, 0x73, 0x74, 0x20, 0x64, 0x79, 0x6e, 0x61, 0x6d, 0x69, 0x63, \
0x20, 0x50, 0x4f, 0x53, 0x54, \
0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, \
0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, \
0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00
static uint16_t test_http_service_port = SERVER_PORT; static uint16_t test_http_service_port = SERVER_PORT;
HTTP_SERVICE_DEFINE(test_http_service, SERVER_IPV4_ADDR, HTTP_SERVICE_DEFINE(test_http_service, SERVER_IPV4_ADDR,
@ -393,21 +409,13 @@ ZTEST(server_function_tests, test_http1_static_get)
"Received data doesn't match expected response"); "Received data doesn't match expected response");
} }
ZTEST(server_function_tests, test_http2_dynamic_post) static void common_verify_http2_dynamic_post_request(const uint8_t *request,
size_t request_len)
{ {
static const uint8_t request_post_dynamic[] = {
TEST_HTTP2_MAGIC,
TEST_HTTP2_SETTINGS,
TEST_HTTP2_SETTINGS_ACK,
TEST_HTTP2_HEADERS_POST_DYNAMIC_STREAM_1,
TEST_HTTP2_DATA_POST_DYNAMIC_STREAM_1,
TEST_HTTP2_GOAWAY,
};
size_t offset = 0; size_t offset = 0;
int ret; int ret;
ret = zsock_send(client_fd, request_post_dynamic, ret = zsock_send(client_fd, request, request_len, 0);
sizeof(request_post_dynamic), 0);
zassert_not_equal(ret, -1, "send() failed (%d)", errno); zassert_not_equal(ret, -1, "send() failed (%d)", errno);
memset(buf, 0, sizeof(buf)); memset(buf, 0, sizeof(buf));
@ -423,6 +431,21 @@ ZTEST(server_function_tests, test_http2_dynamic_post)
dynamic_payload_len, "Wrong dynamic resource data"); dynamic_payload_len, "Wrong dynamic resource data");
} }
ZTEST(server_function_tests, test_http2_dynamic_post)
{
static const uint8_t request_post_dynamic[] = {
TEST_HTTP2_MAGIC,
TEST_HTTP2_SETTINGS,
TEST_HTTP2_SETTINGS_ACK,
TEST_HTTP2_HEADERS_POST_DYNAMIC_STREAM_1,
TEST_HTTP2_DATA_POST_DYNAMIC_STREAM_1,
TEST_HTTP2_GOAWAY,
};
common_verify_http2_dynamic_post_request(request_post_dynamic,
sizeof(request_post_dynamic));
}
ZTEST(server_function_tests, test_http1_dynamic_upgrade_post) ZTEST(server_function_tests, test_http1_dynamic_upgrade_post)
{ {
static const char http1_request[] = static const char http1_request[] =
@ -492,23 +515,16 @@ ZTEST(server_function_tests, test_http1_dynamic_post)
dynamic_payload_len, "Wrong dynamic resource data"); dynamic_payload_len, "Wrong dynamic resource data");
} }
ZTEST(server_function_tests, test_http2_dynamic_get) static void common_verify_http2_dynamic_get_request(const uint8_t *request,
size_t request_len)
{ {
static const uint8_t request_get_dynamic[] = {
TEST_HTTP2_MAGIC,
TEST_HTTP2_SETTINGS,
TEST_HTTP2_SETTINGS_ACK,
TEST_HTTP2_HEADERS_GET_DYNAMIC_STREAM_1,
TEST_HTTP2_GOAWAY,
};
size_t offset = 0; size_t offset = 0;
int ret; int ret;
dynamic_payload_len = strlen(TEST_DYNAMIC_GET_PAYLOAD); dynamic_payload_len = strlen(TEST_DYNAMIC_GET_PAYLOAD);
memcpy(dynamic_payload, TEST_DYNAMIC_GET_PAYLOAD, dynamic_payload_len); memcpy(dynamic_payload, TEST_DYNAMIC_GET_PAYLOAD, dynamic_payload_len);
ret = zsock_send(client_fd, request_get_dynamic, ret = zsock_send(client_fd, request, request_len, 0);
sizeof(request_get_dynamic), 0);
zassert_not_equal(ret, -1, "send() failed (%d)", errno); zassert_not_equal(ret, -1, "send() failed (%d)", errno);
memset(buf, 0, sizeof(buf)); memset(buf, 0, sizeof(buf));
@ -523,6 +539,20 @@ ZTEST(server_function_tests, test_http2_dynamic_get)
HTTP2_FLAG_END_STREAM); HTTP2_FLAG_END_STREAM);
} }
ZTEST(server_function_tests, test_http2_dynamic_get)
{
static const uint8_t request_get_dynamic[] = {
TEST_HTTP2_MAGIC,
TEST_HTTP2_SETTINGS,
TEST_HTTP2_SETTINGS_ACK,
TEST_HTTP2_HEADERS_GET_DYNAMIC_STREAM_1,
TEST_HTTP2_GOAWAY,
};
common_verify_http2_dynamic_get_request(request_get_dynamic,
sizeof(request_get_dynamic));
}
ZTEST(server_function_tests, test_http1_dynamic_upgrade_get) ZTEST(server_function_tests, test_http1_dynamic_upgrade_get)
{ {
static const char http1_request[] = static const char http1_request[] =
@ -647,6 +677,35 @@ ZTEST(server_function_tests, test_http1_connection_close)
zassert_equal(ret, 0, "Connection should've been closed"); zassert_equal(ret, 0, "Connection should've been closed");
} }
ZTEST(server_function_tests, test_http2_post_data_with_padding)
{
static const uint8_t request_post_dynamic[] = {
TEST_HTTP2_MAGIC,
TEST_HTTP2_SETTINGS,
TEST_HTTP2_SETTINGS_ACK,
TEST_HTTP2_HEADERS_POST_DYNAMIC_STREAM_1,
TEST_HTTP2_DATA_POST_DYNAMIC_STREAM_1_PADDED,
TEST_HTTP2_GOAWAY,
};
common_verify_http2_dynamic_post_request(request_post_dynamic,
sizeof(request_post_dynamic));
}
ZTEST(server_function_tests, test_http2_get_headers_with_padding)
{
static const uint8_t request_get_dynamic[] = {
TEST_HTTP2_MAGIC,
TEST_HTTP2_SETTINGS,
TEST_HTTP2_SETTINGS_ACK,
TEST_HTTP2_HEADERS_GET_DYNAMIC_STREAM_1_PADDED,
TEST_HTTP2_GOAWAY,
};
common_verify_http2_dynamic_get_request(request_get_dynamic,
sizeof(request_get_dynamic));
}
ZTEST(server_function_tests_no_init, test_http_server_start_stop) ZTEST(server_function_tests_no_init, test_http_server_start_stop)
{ {
struct sockaddr_in sa = { 0 }; struct sockaddr_in sa = { 0 };