kgiusti commented on a change in pull request #660: DISPATCH-1513 - Fixed router crash when running http requests over ht… URL: https://github.com/apache/qpid-dispatch/pull/660#discussion_r366918885
########## File path: cmake/FindLibWebSockets.cmake ########## @@ -60,6 +60,20 @@ else() LIBWEBSOCKETS DEFAULT_MSG LIBWEBSOCKETS_VERSION_STRING LIBWEBSOCKETS_LIBRARIES LIBWEBSOCKETS_INCLUDE_DIRS) endif() + +if (LIBWEBSOCKETS_FOUND) + # This is a fix for DISPATCH-1513. libwebsockets versions 3.2.0 introduces a new flag called LWS_SERVER_OPTION_ALLOW_HTTP_ON_HTTPS_LISTENER + # The new flag allows (as the flag says) HTTP pver HTTPS listeners. Since this flag is not available before lws 3.2.0 we need + # to selectively include the flag in http-libwebsockets.c + set(OPTION_ALLOW_HTTP_ON_HTTPS_LISTENER "") + set(TEST_OPTION_ALLOW_HTTP_ON_HTTPS_LISTENER "#") + set(LWS_VERSION_WITH_SSL_FIX "3.2.0") + if (LIBWEBSOCKETS_VERSION_STRING AND (LIBWEBSOCKETS_VERSION_STRING VERSION_GREATER_EQUAL LWS_VERSION_WITH_SSL_FIX)) + set(OPTION_ALLOW_HTTP_ON_HTTPS_LISTENER "| LWS_SERVER_OPTION_ALLOW_HTTP_ON_HTTPS_LISTENER") Review comment: Can you instead check the libwebsockets version directly in http-libwebsockets.c? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org