Build time patches to nginx spdy patch #52
Kaj Niemi
kajtzu at a51.org
Fri Aug 24 09:46:42 UTC 2012
Hi,
SPDY patch #52 is missing a few conditionals that are required when the patch has been applied but nginx is built using --without-ssl_spdy_module (on RHEL6 I need to build one version of nginx against an openssl that has it (ours) and another without (vendor's)).
Thanks.
--- src/http/ngx_http_request.c.old 2012-08-24 12:21:14.377350628 +0300
+++ src/http/ngx_http_request.c 2012-08-24 12:21:17.774312838 +0300
@@ -1981,10 +1981,12 @@
ngx_http_request_t *pr;
ngx_http_core_loc_conf_t *clcf;
+#if (NGX_HTTP_SPDY)
if (r->spdy_stream) {
ngx_http_spdy_finalize_request(r, rc);
return;
}
+#endif
c = r->connection;
--- src/http/ngx_http_request_body.c.old 2012-08-24 12:27:26.553322219 +0300
+++ src/http/ngx_http_request_body.c 2012-08-24 12:28:58.491538266 +0300
@@ -471,10 +471,12 @@
return NGX_OK;
}
+#if (NGX_HTTP_SPDY)
if (r->spdy_stream) {
r->discard_body = 1;
return NGX_OK;
}
+#endif
if (ngx_http_test_expect(r) != NGX_OK) {
return NGX_HTTP_INTERNAL_SERVER_ERROR;
--- auto/modules.old 2012-08-24 12:33:19.266063093 +0300
+++ auto/modules 2012-08-24 12:35:23.397878682 +0300
@@ -117,9 +117,13 @@
HTTP_FILTER_MODULES="$HTTP_WRITE_FILTER_MODULE \
$HTTP_HEADER_FILTER_MODULE \
- $HTTP_CHUNKED_FILTER_MODULE \
- $HTTP_SPDY_FILTER_MODULE \
- $HTTP_RANGE_HEADER_FILTER_MODULE"
+ $HTTP_CHUNKED_FILTER_MODULE"
+
+if [ $HTTP_SPDY = YES ]; then
+ HTTP_FILTER_MODULES="$HTTP_FILTER_MODULES $HTTP_SPDY_FILTER_MODULE"
+fi
+
+HTTP_FILTER_MODULES="$HTTP_FILTER_MODULES $HTTP_RANGE_HEADER_FILTER_MODULE"
if [ $HTTP_GZIP = YES ]; then
have=NGX_HTTP_GZIP . auto/have
Kaj
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4383 bytes
Desc: not available
URL: <http://mailman.nginx.org/pipermail/nginx-devel/attachments/20120824/9ed33fdb/attachment.bin>
More information about the nginx-devel
mailing list