Skip to content

sslh: update to 2.3.1, add libproxyprotocol support#28372

Open
meiser79 wants to merge 3 commits intoopenwrt:masterfrom
meiser79:sslh-proxyprotocol
Open

sslh: update to 2.3.1, add libproxyprotocol support#28372
meiser79 wants to merge 3 commits intoopenwrt:masterfrom
meiser79:sslh-proxyprotocol

Conversation

@meiser79
Copy link
Copy Markdown

See https://rutschle.net/tech/sslh/doc/proxyprotocol

📦 Package Details

Maintainer: @neheb

Description:
Add libproxysupport to sslh by adding the libproxysupport library and compile sslh against it.

🧪 Run Testing Details

  • **OpenWrt Version: master
  • OpenWrt Target/Subtarget:
  • **OpenWrt Device: x86_64

✅ Formalities

  • I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

@efahl
Copy link
Copy Markdown
Contributor

efahl commented Jan 24, 2026

@meiser79 Seems like this is three PRs all mashed into one. Might be easier to get reviews and commits if you did it in pieces:

  • libproxyprotocol: add new package
  • sslh: update to 2.3.0
  • sslh: add proxyprotocol support

@efahl
Copy link
Copy Markdown
Contributor

efahl commented Jan 24, 2026

Ah, yes, that's much better. It shows that the patches are related to the version update and not the addition of proxyprotocol.

@meiser79
Copy link
Copy Markdown
Author

libproxyprotocol file size:

root@openwrt:~# ll /usr/lib/libproxyprotocol.so 
-rwxr-xr-x    1 root     root         24578 Dec 22 12:52 /usr/lib/libproxyprotocol.so*

@efahl
Copy link
Copy Markdown
Contributor

efahl commented Jan 24, 2026

I'm guessing the formalities will fail on this due to no messages in the commits. And that size measurement would be a good tidbit to add to the message:

 libproxyprotocol: add new package 

Add new package to support blah blah blah

Resulting library size is
# ll /usr/lib/libproxyprotocol.so 
-rwxr-xr-x    1 root     root         24578 Dec 22 12:52 /usr/lib/libproxyprotocol.so*

Signed-off-by: ...

@meiser79 meiser79 changed the title sslh: update to 2.3.0, add libproxyprotocol support sslh: update to 2.3.1, add libproxyprotocol support Apr 3, 2026
@meiser79
Copy link
Copy Markdown
Author

meiser79 commented Apr 3, 2026

@efahl, may I kindly ask to review again? Could it be merged?

@efahl
Copy link
Copy Markdown
Contributor

efahl commented Apr 3, 2026

The resulting code looks ok, but the commit history is problematic and needs to be squashed.

This should probably have just two commits: the first to add the new libproxyprotocol package, second one to extend sslh using it. Those intermediate ones that add and then remove patches just obfuscate what's going on.

@meiser79 meiser79 force-pushed the sslh-proxyprotocol branch from ae40dff to 2b05af4 Compare April 3, 2026 13:56
@meiser79
Copy link
Copy Markdown
Author

meiser79 commented Apr 3, 2026

@efahl, thanks, cleanup done.

meiser79 added 3 commits April 3, 2026 16:04
Add a new package which includes a library for PROXY protocol support.
According to the GitHub project, it's "an ANSI C library to parse and
create PROXY protocol v1 and v2 headers with support for all
TLVs including the custom ones from AWS and Azure."

Resulting library size is
-rwxr-xr-x    1 root     root         24578 Dec 22 12:52 /usr/lib/libproxyprotocol.so*

Signed-off-by: Peter Meiser <peter.meiser@gmx.com>
Signed-off-by: Peter Meiser <peter.meiser@gmx.com>
@meiser79 meiser79 force-pushed the sslh-proxyprotocol branch from 2b05af4 to 6c304f7 Compare April 3, 2026 14:04
@efahl
Copy link
Copy Markdown
Contributor

efahl commented Apr 3, 2026

Looks good to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants