CrowCpp / Crow / 851

Committed 17 Nov 2025 11:09AM UTC coverage: 87.46% (-0.06%) from 87.521%

Build # 851

Build Type

push

gh-actions

Committed by

gittiver

Commit Message

readded condition for sending_buffers (seems get lost on last refactoring)

Run Details

22 of 24 new or added lines in 1 file covered. (91.67%)

3 existing lines in 1 file now uncovered.

4129 of 4721 relevant lines covered (87.46%)

273.41 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

96.84

/include/crow/parser.h

#pragma once

#include <string>
#include <unordered_map>
#include <algorithm>

#include "crow/http_request.h"
#include "crow/http_parser_merged.h"

namespace crow
{
    /// A wrapper for `nodejs/http-parser`.

    ///
    /// Used to generate a \ref crow.request from the TCP socket buffer.
    template<typename Handler>
    struct HTTPParser : public http_parser
    {
        static int on_message_begin(http_parser*)
        {
            return 0;
        }
        static int on_method(http_parser* self_)
        {
            HTTPParser* self = static_cast<HTTPParser*>(self_);
            self->req.method = static_cast<HTTPMethod>(self->method);

            return 0;
        }
        static int on_url(http_parser* self_, const char* at, size_t length)
        {
            HTTPParser* self = static_cast<HTTPParser*>(self_);
            self->req.raw_url.insert(self->req.raw_url.end(), at, at + length);
            self->req.url_params = query_string(self->req.raw_url);
            self->req.url = self->req.raw_url.substr(0, self->qs_point != 0 ? self->qs_point : std::string::npos);

            self->process_url();

            return 0;
        }
        static int on_header_field(http_parser* self_, const char* at, size_t length)
        {
            HTTPParser* self = static_cast<HTTPParser*>(self_);
            switch (self->header_building_state)
            {
                case 0:
                    if (!self->header_value.empty())
                    {
                        self->req.headers.emplace(std::move(self->header_field), std::move(self->header_value));
                    }
                    self->header_field.assign(at, at + length);
                    self->header_building_state = 1;
                    break;
                case 1:
                    self->header_field.insert(self->header_field.end(), at, at + length);
                    break;
            }
            return 0;
        }
        static int on_header_value(http_parser* self_, const char* at, size_t length)
        {
            HTTPParser* self = static_cast<HTTPParser*>(self_);
            switch (self->header_building_state)
            {
                case 0:
                    self->header_value.insert(self->header_value.end(), at, at + length);
                    break;
                case 1:
                    self->header_building_state = 0;
                    self->header_value.assign(at, at + length);
                    break;
            }
            return 0;
        }
        static int on_headers_complete(http_parser* self_)
        {
            HTTPParser* self = static_cast<HTTPParser*>(self_);
            if (!self->header_field.empty())
            {
                self->req.headers.emplace(std::move(self->header_field), std::move(self->header_value));
            }

            self->set_connection_parameters();

            self->process_header();
            return 0;
        }
        static int on_body(http_parser* self_, const char* at, size_t length)
        {
            HTTPParser* self = static_cast<HTTPParser*>(self_);
            self->req.body.insert(self->req.body.end(), at, at + length);
            return 0;
        }
        static int on_message_complete(http_parser* self_)
        {
            HTTPParser* self = static_cast<HTTPParser*>(self_);

            self->message_complete = true;
            self->process_message();
            return 0;
        }
        HTTPParser(Handler* handler):
          http_parser(),
          handler_(handler)
        {
            http_parser_init(this);
        }

        // return false on error
        /// Parse a buffer into the different sections of an HTTP request.
        bool feed(const char* buffer, int length)
        {
            if (message_complete)
                return true;

            const static http_parser_settings settings_{
              on_message_begin,
              on_method,
              on_url,
              on_header_field,
              on_header_value,
              on_headers_complete,
              on_body,
              on_message_complete,
            };

            int nparsed = http_parser_execute(this, &settings_, buffer, length);
            if (http_errno != CHPE_OK)
            {
                return false;
            }
            return nparsed == length;
        }

        bool done()
        {
            return feed(nullptr, 0);
        }

        void clear()
        {
            req = crow::request();
            header_field.clear();
            header_value.clear();
            header_building_state = 0;
            qs_point = 0;
            message_complete = false;
            state = CROW_NEW_MESSAGE();
        }

        inline void process_url()
        {
            handler_->handle_url();
        }

        inline void process_header()
        {
            handler_->handle_header();
        }

        inline void process_message()
        {
            handler_->handle();
        }

        inline void set_connection_parameters()
        {
            req.http_ver_major = http_major;
            req.http_ver_minor = http_minor;

            //NOTE(EDev): it seems that the problem is with crow's policy on closing the connection for HTTP_VERSION < 1.0, the behaviour for that in crow is "don't close the connection, but don't send a keep-alive either"

            // HTTP1.1 = always send keep_alive, HTTP1.0 = only send if header exists, HTTP?.? = never send
            req.keep_alive = (http_major == 1 && http_minor == 0) ?
                               ((flags & F_CONNECTION_KEEP_ALIVE) ? true : false) :
                               ((http_major == 1 && http_minor == 1) ? true : false);

            // HTTP1.1 = only close if close header exists, HTTP1.0 = always close unless keep_alive header exists, HTTP?.?= never close
            req.close_connection = (http_major == 1 && http_minor == 0) ?
                                     ((flags & F_CONNECTION_KEEP_ALIVE) ? false : true) :
                                     ((http_major == 1 && http_minor == 1) ? ((flags & F_CONNECTION_CLOSE) ? true : false) : false);
            req.upgrade = static_cast<bool>(upgrade);
        }

        /// The final request that this parser outputs.
        ///
        /// Data parsed is put directly into this object as soon as the related callback returns. (e.g. the request will have the cooorect method as soon as on_method() returns)
        request req;

    private:
        int header_building_state = 0;
        bool message_complete = false;
        std::string header_field;
        std::string header_value;

        Handler* handler_; ///< This is currently an HTTP connection object (\ref crow.Connection).
    };
} // namespace crow

#undef CROW_NEW_MESSAGE
#undef CROW_start_state

1	#pragma once
2
3	#include <string>
4	#include <unordered_map>
5	#include <algorithm>
6
7	#include "crow/http_request.h"
8	#include "crow/http_parser_merged.h"
9
10	namespace crow
11	{
12	/// A wrapper for `nodejs/http-parser`.
13
14	///
15	/// Used to generate a \ref crow.request from the TCP socket buffer.
16	template<typename Handler>
17	struct HTTPParser : public http_parser
18	{
19	static int on_message_begin(http_parser*)	172✔
20	{
21	return 0;	172✔
22	}
23	static int on_method(http_parser* self_)	544✔
24	{
25	HTTPParser* self = static_cast<HTTPParser*>(self_);	544✔
26	self->req.method = static_cast<HTTPMethod>(self->method);	544✔
27
28	return 0;	544✔
29	}
30	static int on_url(http_parser* self_, const char* at, size_t length)	168✔
31	{
32	HTTPParser* self = static_cast<HTTPParser*>(self_);	168✔
33	self->req.raw_url.insert(self->req.raw_url.end(), at, at + length);	168✔
34	self->req.url_params = query_string(self->req.raw_url);	168✔
35	self->req.url = self->req.raw_url.substr(0, self->qs_point != 0 ? self->qs_point : std::string::npos);	168✔
36
37	self->process_url();	168✔
38
39	return 0;	168✔
40	}
41	static int on_header_field(http_parser* self_, const char* at, size_t length)	212✔
42	{
43	HTTPParser* self = static_cast<HTTPParser*>(self_);	212✔
44	switch (self->header_building_state)	212✔
45	{
46	case 0:	208✔
47	if (!self->header_value.empty())	208✔
48	{
49	self->req.headers.emplace(std::move(self->header_field), std::move(self->header_value));	102✔
50	}
51	self->header_field.assign(at, at + length);	208✔
52	self->header_building_state = 1;	208✔
53	break;	208✔
54	case 1:	4✔
55	self->header_field.insert(self->header_field.end(), at, at + length);	4✔
56	break;	4✔
57	}
58	return 0;	212✔
59	}
60	static int on_header_value(http_parser* self_, const char* at, size_t length)	208✔
61	{
62	HTTPParser* self = static_cast<HTTPParser*>(self_);	208✔
63	switch (self->header_building_state)	208✔
64	{
UNCOV 65	case 0:	×
UNCOV 66	self->header_value.insert(self->header_value.end(), at, at + length);	×
UNCOV 67	break;	×
68	case 1:	208✔
69	self->header_building_state = 0;	208✔
70	self->header_value.assign(at, at + length);	208✔
71	break;	208✔
72	}
73	return 0;	208✔
74	}
75	static int on_headers_complete(http_parser* self_)	162✔
76	{
77	HTTPParser* self = static_cast<HTTPParser*>(self_);	162✔
78	if (!self->header_field.empty())	162✔
79	{
80	self->req.headers.emplace(std::move(self->header_field), std::move(self->header_value));	106✔
81	}
82
83	self->set_connection_parameters();	162✔
84
85	self->process_header();	162✔
86	return 0;	162✔
87	}
88	static int on_body(http_parser* self_, const char* at, size_t length)	6✔
89	{
90	HTTPParser* self = static_cast<HTTPParser*>(self_);	6✔
91	self->req.body.insert(self->req.body.end(), at, at + length);	6✔
92	return 0;	6✔
93	}
94	static int on_message_complete(http_parser* self_)	162✔
95	{
96	HTTPParser* self = static_cast<HTTPParser*>(self_);	162✔
97
98	self->message_complete = true;	162✔
99	self->process_message();	162✔
100	return 0;	162✔
101	}
102	HTTPParser(Handler* handler):	206✔
103	http_parser(),
104	handler_(handler)	206✔
105	{
106	http_parser_init(this);	206✔
107	}	206✔
108
109	// return false on error
110	/// Parse a buffer into the different sections of an HTTP request.
111	bool feed(const char* buffer, int length)	322✔
112	{
113	if (message_complete)	322✔
114	return true;	18✔
115
116	const static http_parser_settings settings_{
117	on_message_begin,
118	on_method,
119	on_url,
120	on_header_field,
121	on_header_value,
122	on_headers_complete,
123	on_body,
124	on_message_complete,
125	};
126
127	int nparsed = http_parser_execute(this, &settings_, buffer, length);	304✔
128	if (http_errno != CHPE_OK)	304✔
129	{
130	return false;	26✔
131	}
132	return nparsed == length;	278✔
133	}
134
135	bool done()	138✔
136	{
137	return feed(nullptr, 0);	138✔
138	}
139
140	void clear()	442✔
141	{
142	req = crow::request();	442✔
143	header_field.clear();	442✔
144	header_value.clear();	442✔
145	header_building_state = 0;	442✔
146	qs_point = 0;	442✔
147	message_complete = false;	442✔
148	state = CROW_NEW_MESSAGE();	442✔
149	}	442✔
150
151	inline void process_url()	168✔
152	{
153	handler_->handle_url();	168✔
154	}	168✔
155
156	inline void process_header()	162✔
157	{
158	handler_->handle_header();	162✔
159	}	162✔
160
161	inline void process_message()	162✔
162	{
163	handler_->handle();	162✔
164	}	162✔
165
166	inline void set_connection_parameters()	162✔
167	{
168	req.http_ver_major = http_major;	162✔
169	req.http_ver_minor = http_minor;	162✔
170
171	//NOTE(EDev): it seems that the problem is with crow's policy on closing the connection for HTTP_VERSION < 1.0, the behaviour for that in crow is "don't close the connection, but don't send a keep-alive either"
172
173	// HTTP1.1 = always send keep_alive, HTTP1.0 = only send if header exists, HTTP?.? = never send
174	req.keep_alive = (http_major == 1 && http_minor == 0) ?	310✔
175	((flags & F_CONNECTION_KEEP_ALIVE) ? true : false) :	14✔
176	((http_major == 1 && http_minor == 1) ? true : false);	148✔
177
178	// HTTP1.1 = only close if close header exists, HTTP1.0 = always close unless keep_alive header exists, HTTP?.?= never close
179	req.close_connection = (http_major == 1 && http_minor == 0) ?	310✔
180	((flags & F_CONNECTION_KEEP_ALIVE) ? false : true) :	14✔
181	((http_major == 1 && http_minor == 1) ? ((flags & F_CONNECTION_CLOSE) ? true : false) : false);	148✔
182	req.upgrade = static_cast<bool>(upgrade);	162✔
183	}	162✔
184
185	/// The final request that this parser outputs.
186	///
187	/// Data parsed is put directly into this object as soon as the related callback returns. (e.g. the request will have the cooorect method as soon as on_method() returns)
188	request req;
189
190	private:
191	int header_building_state = 0;
192	bool message_complete = false;
193	std::string header_field;
194	std::string header_value;
195
196	Handler* handler_; ///< This is currently an HTTP connection object (\ref crow.Connection).
197	};
198	} // namespace crow
199
200	#undef CROW_NEW_MESSAGE
201	#undef CROW_start_state

CrowCpp / Crow / 851

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous