[PR PATCH] nginx: add support for stream proxying

[PR PATCH] nginx: add support for stream proxying

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 644 bytes --]

There is a new pull request by ahesford against master on the void-packages repository

https://github.com/ahesford/void-packages nginx
https://github.com/void-linux/void-packages/pull/18474

nginx: add support for stream proxying
Nginx supports, with the "stream" family of modules, acting as a (reverse) proxy for arbitrary data streams. This PR adds a "stream" build option, enabled by default, that will configure nginx for stream proxying. Offloading encryption responsibilities to an nginx strem proxy represents a good use case for this feature.

A patch file from https://github.com/void-linux/void-packages/pull/18474.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-nginx-18474.patch --]
[-- Type: text/x-diff, Size: 1705 bytes --]

From 5b49dab6659a704f0d03780948a96c5f568c9969 Mon Sep 17 00:00:00 2001
From: "Andrew J. Hesford" <ajh@sideband.org>
Date: Wed, 22 Jan 2020 11:20:09 -0500
Subject: [PATCH] nginx: add support for stream proxying

---
 srcpkgs/nginx/template | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/srcpkgs/nginx/template b/srcpkgs/nginx/template
index f8b04e6df6c..374b6fe6c1d 100644
--- a/srcpkgs/nginx/template
+++ b/srcpkgs/nginx/template
@@ -1,8 +1,8 @@
 # Template file for 'nginx'
-reverts="1.17.7_1"
 pkgname=nginx
+reverts="1.17.7_1"
 version=1.16.1
-revision=3
+revision=4
 build_style=gnu-makefile
 hostmakedepends="libressl-devel pcre-devel $(vopt_if geoip geoip-devel)"
 makedepends="${hostmakedepends}"
@@ -32,8 +32,8 @@ make_dirs="/var/log/nginx 0750 root root
  /var/tmp 1777 root root
  /var/tmp/nginx 0750 nginx root"
 
-build_options="geoip"
-build_options_default="geoip"
+build_options="geoip stream"
+build_options_default="geoip stream"
 
 do_configure() {
 	local cfgdir=/etc/nginx
@@ -44,6 +44,14 @@ do_configure() {
 		unset CC CPP LD CFLAGS CPPFLAGS LDFLAGS
 	fi
 
+	local stream_modules="\
+		--with-stream \
+		--with-stream_realip_module \
+		--with-stream_ssl_module \
+		--with-stream_ssl_preread_module \
+		$(vopt_if geoip --with-stream_geoip_module) \
+	"
+
 	./configure --prefix=${cfgdir} \
 		--conf-path=${cfgdir}/nginx.conf \
 		--sbin-path=/usr/bin/nginx \
@@ -67,7 +75,8 @@ do_configure() {
 		--with-http_ssl_module \
 		--with-http_stub_status_module \
 		--with-http_sub_module \
-		$(vopt_if geoip --with-http_geoip_module)
+		$(vopt_if geoip --with-http_geoip_module) \
+		$(vopt_if stream "$stream_modules")
 }
 
 pre_build() {

Re: [PR PATCH] [Updated] nginx: add support for stream proxying

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 649 bytes --]

There is an updated pull request by ahesford against master on the void-packages repository

https://github.com/ahesford/void-packages nginx
https://github.com/void-linux/void-packages/pull/18474

nginx: add support for stream proxying
Nginx supports, with the "stream" family of modules, acting as a (reverse) proxy for arbitrary data streams. This PR adds a "stream" build option, enabled by default, that will configure nginx for stream proxying. Offloading encryption responsibilities to an nginx strem proxy represents a good use case for this feature.

A patch file from https://github.com/void-linux/void-packages/pull/18474.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-nginx-18474.patch --]
[-- Type: text/x-diff, Size: 4178 bytes --]

From aae0dc5c5b6f959618e8f590458faad6500190c3 Mon Sep 17 00:00:00 2001
From: "Andrew J. Hesford" <ajh@sideband.org>
Date: Wed, 22 Jan 2020 11:20:09 -0500
Subject: [PATCH] nginx: add support for stream proxying

---
 srcpkgs/nginx/files/ngx_auto_config.h.armv6l | 14 ++++++++++++--
 srcpkgs/nginx/template                       | 20 +++++++++++++++-----
 2 files changed, 27 insertions(+), 7 deletions(-)

diff --git a/srcpkgs/nginx/files/ngx_auto_config.h.armv6l b/srcpkgs/nginx/files/ngx_auto_config.h.armv6l
index 6ee9d6eb213..f32af78d12b 100644
--- a/srcpkgs/nginx/files/ngx_auto_config.h.armv6l
+++ b/srcpkgs/nginx/files/ngx_auto_config.h.armv6l
@@ -1,7 +1,7 @@
-#define NGX_CONFIGURE " --prefix=/etc/nginx --conf-path=/etc/nginx/nginx.conf --sbin-path=/usr/bin/nginx --pid-path=/run/nginx.pid --lock-path=/var/lock/nginx.lock --user=nginx --group=nginx --http-log-path=/var/log/nginx/access.log --error-log-path=/var/log/nginx/error.log --http-client-body-temp-path=/var/tmp/nginx/client-body --http-proxy-temp-path=/var/tmp/nginx/proxy --http-fastcgi-temp-path=/var/tmp/nginx/fastcgi --http-scgi-temp-path=/var/tmp/nginx/scgi --http-uwsgi-temp-path=/var/tmp/nginx/uwsgi --with-mail --with-mail_ssl_module --with-pcre-jit --with-file-aio --with-http_gunzip_module --with-http_v2_module --with-http_dav_module --with-http_gzip_static_module --with-http_realip_module --with-http_ssl_module --with-http_stub_status_module --with-http_geoip_module"
+#define NGX_CONFIGURE " --prefix=/etc/nginx --conf-path=/etc/nginx/nginx.conf --sbin-path=/usr/bin/nginx --pid-path=/run/nginx.pid --lock-path=/var/lock/nginx.lock --user=nginx --group=nginx --http-log-path=/var/log/nginx/access.log --error-log-path=/var/log/nginx/error.log --http-client-body-temp-path=/var/tmp/nginx/client-body --http-proxy-temp-path=/var/tmp/nginx/proxy --http-fastcgi-temp-path=/var/tmp/nginx/fastcgi --http-scgi-temp-path=/var/tmp/nginx/scgi --http-uwsgi-temp-path=/var/tmp/nginx/uwsgi --with-mail --with-mail_ssl_module --with-pcre-jit --with-file-aio --with-http_gunzip_module --with-http_v2_module --with-http_dav_module --with-http_gzip_static_module --with-http_realip_module --with-http_ssl_module --with-http_stub_status_module --with-http_sub_module --with-http_geoip_module --with-stream --with-stream_realip_module --with-stream_ssl_module --with-stream_ssl_preread_module --with-stream_geoip_module"
 
 #ifndef NGX_COMPILER
-#define NGX_COMPILER  "gcc 8.2.0 (GCC) "
+#define NGX_COMPILER  "gcc 9.2.0 (GCC) "
 #endif
 
 
@@ -443,6 +443,16 @@
 #endif
 
 
+#ifndef NGX_STREAM_SSL
+#define NGX_STREAM_SSL  1
+#endif
+
+
+#ifndef NGX_STREAM_UPSTREAM_ZONE
+#define NGX_STREAM_UPSTREAM_ZONE  1
+#endif
+
+
 #ifndef NGX_PCRE
 #define NGX_PCRE  1
 #endif
diff --git a/srcpkgs/nginx/template b/srcpkgs/nginx/template
index f8b04e6df6c..f0d96046d7a 100644
--- a/srcpkgs/nginx/template
+++ b/srcpkgs/nginx/template
@@ -1,8 +1,8 @@
 # Template file for 'nginx'
-reverts="1.17.7_1"
 pkgname=nginx
+reverts="1.17.7_1"
 version=1.16.1
-revision=3
+revision=4
 build_style=gnu-makefile
 hostmakedepends="libressl-devel pcre-devel $(vopt_if geoip geoip-devel)"
 makedepends="${hostmakedepends}"
@@ -32,8 +32,9 @@ make_dirs="/var/log/nginx 0750 root root
  /var/tmp 1777 root root
  /var/tmp/nginx 0750 nginx root"
 
-build_options="geoip"
-build_options_default="geoip"
+build_options="geoip stream"
+desc_option_stream="Enable support for stream proxies"
+build_options_default="geoip stream"
 
 do_configure() {
 	local cfgdir=/etc/nginx
@@ -44,6 +45,14 @@ do_configure() {
 		unset CC CPP LD CFLAGS CPPFLAGS LDFLAGS
 	fi
 
+	local stream_modules="\
+		--with-stream \
+		--with-stream_realip_module \
+		--with-stream_ssl_module \
+		--with-stream_ssl_preread_module \
+		$(vopt_if geoip --with-stream_geoip_module) \
+	"
+
 	./configure --prefix=${cfgdir} \
 		--conf-path=${cfgdir}/nginx.conf \
 		--sbin-path=/usr/bin/nginx \
@@ -67,7 +76,8 @@ do_configure() {
 		--with-http_ssl_module \
 		--with-http_stub_status_module \
 		--with-http_sub_module \
-		$(vopt_if geoip --with-http_geoip_module)
+		$(vopt_if geoip --with-http_geoip_module) \
+		$(vopt_if stream "$stream_modules")
 }
 
 pre_build() {

Re: nginx: add support for stream proxying

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 229 bytes --]

New comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/18474#issuecomment-577332702

Comment:
Oops, I forgot to deal with ARM when making these changes. Everything should build now.

Re: [PR PATCH] [Merged]: nginx: add support for stream proxying

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 493 bytes --]

There's a merged pull request on the void-packages repository

nginx: add support for stream proxying
https://github.com/void-linux/void-packages/pull/18474

Description:
Nginx supports, with the "stream" family of modules, acting as a (reverse) proxy for arbitrary data streams. This PR adds a "stream" build option, enabled by default, that will configure nginx for stream proxying. Offloading encryption responsibilities to an nginx strem proxy represents a good use case for this feature.