> Am 16.10.2023 um 14:28 schrieb jorton@apache.org:
>
> Author: jorton
> Date: Mon Oct 16 12:28:13 2023
> New Revision: 1913019
>
> URL: http://svn.apache.org/viewvc?rev=1913019&view=rev
> Log:
> Further h2 compile fixes:
>
> * modules/http2/h2_session.c (h2_session_start):
> Restrict WebSockets options handling to with-WS builds.
>
> * modules/http2/h2_ws.c: Don't include apr_encode.h (not used).
>
> Modified:
> httpd/httpd/trunk/modules/http2/h2_session.c
> httpd/httpd/trunk/modules/http2/h2_ws.c
>
> Modified: httpd/httpd/trunk/modules/http2/h2_session.c
> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http2/h2_session.c?rev=1913019&r1=1913018&r2=1913019&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/modules/http2/h2_session.c (original)
> +++ httpd/httpd/trunk/modules/http2/h2_session.c Mon Oct 16 12:28:13 2023
> @@ -1085,11 +1085,13 @@ static apr_status_t h2_session_start(h2_
> settings[slen].value = win_size;
> ++slen;
> }
> +#if H2_USE_WEBSOCKETS
> if (h2_config_sgeti(session->s, H2_CONF_WEBSOCKETS)) {
> settings[slen].settings_id = NGHTTP2_SETTINGS_ENABLE_CONNECT_PROTOCOL;
> settings[slen].value = 1;
> ++slen;
> }
> +#endif
>
> ap_log_cerror(APLOG_MARK, APLOG_DEBUG, status, session->c1,
> H2_SSSN_LOG(APLOGNO(03201), session,
>
Fine.
> Modified: httpd/httpd/trunk/modules/http2/h2_ws.c
> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http2/h2_ws.c?rev=1913019&r1=1913018&r2=1913019&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/modules/http2/h2_ws.c (original)
> +++ httpd/httpd/trunk/modules/http2/h2_ws.c Mon Oct 16 12:28:13 2023
> @@ -19,7 +19,6 @@
> #include "apr.h"
> #include "apr_strings.h"
> #include "apr_lib.h"
> -#include "apr_encode.h"
> #include "apr_sha1.h"
> #include "apr_strmatch.h"
Not working here:
h2_ws.c:70:12: error: call to undeclared function 'apr_pencode_base64_binary'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
return apr_pencode_base64_binary(c->pool, dgst, sizeof(dgst),
^
h2_ws.c:71:38: error: use of undeclared identifier 'APR_ENCODE_NONE'
APR_ENCODE_NONE, NULL);
^
h2_ws.c:123:18: error: call to undeclared function 'apr_pencode_base64_binary'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
key_base64 = apr_pencode_base64_binary(c2->pool, key_raw, sizeof(key_raw),
^
h2_ws.c:124:44: error: use of undeclared identifier 'APR_ENCODE_NONE'
APR_ENCODE_NONE, NULL);
>
> Author: jorton
> Date: Mon Oct 16 12:28:13 2023
> New Revision: 1913019
>
> URL: http://svn.apache.org/viewvc?rev=1913019&view=rev
> Log:
> Further h2 compile fixes:
>
> * modules/http2/h2_session.c (h2_session_start):
> Restrict WebSockets options handling to with-WS builds.
>
> * modules/http2/h2_ws.c: Don't include apr_encode.h (not used).
>
> Modified:
> httpd/httpd/trunk/modules/http2/h2_session.c
> httpd/httpd/trunk/modules/http2/h2_ws.c
>
> Modified: httpd/httpd/trunk/modules/http2/h2_session.c
> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http2/h2_session.c?rev=1913019&r1=1913018&r2=1913019&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/modules/http2/h2_session.c (original)
> +++ httpd/httpd/trunk/modules/http2/h2_session.c Mon Oct 16 12:28:13 2023
> @@ -1085,11 +1085,13 @@ static apr_status_t h2_session_start(h2_
> settings[slen].value = win_size;
> ++slen;
> }
> +#if H2_USE_WEBSOCKETS
> if (h2_config_sgeti(session->s, H2_CONF_WEBSOCKETS)) {
> settings[slen].settings_id = NGHTTP2_SETTINGS_ENABLE_CONNECT_PROTOCOL;
> settings[slen].value = 1;
> ++slen;
> }
> +#endif
>
> ap_log_cerror(APLOG_MARK, APLOG_DEBUG, status, session->c1,
> H2_SSSN_LOG(APLOGNO(03201), session,
>
Fine.
> Modified: httpd/httpd/trunk/modules/http2/h2_ws.c
> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http2/h2_ws.c?rev=1913019&r1=1913018&r2=1913019&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/modules/http2/h2_ws.c (original)
> +++ httpd/httpd/trunk/modules/http2/h2_ws.c Mon Oct 16 12:28:13 2023
> @@ -19,7 +19,6 @@
> #include "apr.h"
> #include "apr_strings.h"
> #include "apr_lib.h"
> -#include "apr_encode.h"
> #include "apr_sha1.h"
> #include "apr_strmatch.h"
Not working here:
h2_ws.c:70:12: error: call to undeclared function 'apr_pencode_base64_binary'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
return apr_pencode_base64_binary(c->pool, dgst, sizeof(dgst),
^
h2_ws.c:71:38: error: use of undeclared identifier 'APR_ENCODE_NONE'
APR_ENCODE_NONE, NULL);
^
h2_ws.c:123:18: error: call to undeclared function 'apr_pencode_base64_binary'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
key_base64 = apr_pencode_base64_binary(c2->pool, key_raw, sizeof(key_raw),
^
h2_ws.c:124:44: error: use of undeclared identifier 'APR_ENCODE_NONE'
APR_ENCODE_NONE, NULL);