Problem is that this PR introduces major performance regressions. Logging must not be added with a cost

How about putting a macro around logging like this?

#if UWS_LOG_LEVEL <= 1
log("No handler for route " + std::string(httpRequest->getMethod()) + " " + std::string(httpRequest->getUrl()), 1);
#endif

The default value of UWS_LOG_LEVEL would be 0. That would let the user decide on the trade-off between performance and debuggability. spdlog does something similar.

Macro would work, but I don't want to put a bunch of text in the code - I would rather have it like so:

One header file contains all text -> you can either include it before App.h and then get logging enabled (or, let's say have WITH_LOG=1 build flag or something).

In the header you have macros like UWS_LOG_REQUEST(req) as functions, so that the calling (source code) remains clutter-free.

But it doesn't have to be C-style macros, you can use constexpr C++17 functions I mean you can have these "macros" as regular C++ code, but have an if constxpr (loglevel < 1) {

}

Also, the logging has to be faster than just concatenating strings like that - use a pre-allocated array and memcpy or similar. One pre-allocated array per thread so mark it thread_local

I have changed the code to fulfill your requirements:

• String concatenations are as efficient as possible
• One pre-allocated log buffer per thread
• No function call is made at all if the verbosity does not allow for messages of that loglevel
• Using the logger is a one-liner

^{It seemed overly complicated to me, but hey! My first real world problem solved with variadic templates.}

I also made use of if constexpr. However a macro is still required, as parameters cannot be constexpr in C++17 and we need logLevel as constexpr. The only way around this would be a template argument, I think, but I don't see how this is cleaner.

One header file contains all text

Not sure if I understand you correctly, here. You want to move all log strings into a separate file like this?

const std::string noHandlerForRoute = "No handler for route ";
const std::string space = " ";
// ...
const std::string returningWithoutResponding = "Error: Returning from a request handler without responding or attaching an abort handler is forbidden!";

How would that be better?

@alexhultman Could you please leave a comment on the changes that I've made? Thanks.

#1132

I added some more logging, including the case when us_create_socket_context fails. However, we cannot log any SSL specific stuff here, as this is handled by uSockets.

Then maybe logging should be part of uSockets

@alexhultman Do we still need logging in uSockets, now that you added an error handler for SSL? And if yes, does it have to be C? If it's not C++, I'm out.

I might have some time in the next days.