sslh: update to 2.3.1, add libproxyprotocol support#28372
sslh: update to 2.3.1, add libproxyprotocol support#28372meiser79 wants to merge 3 commits intoopenwrt:masterfrom
Conversation
|
@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:
|
e47147d to
d691e84
Compare
|
Ah, yes, that's much better. It shows that the patches are related to the version update and not the addition of proxyprotocol. |
|
libproxyprotocol file size: |
|
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: |
d691e84 to
bfbac91
Compare
bfbac91 to
e10ccee
Compare
|
@efahl, may I kindly ask to review again? Could it be merged? |
|
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. |
ae40dff to
2b05af4
Compare
|
@efahl, thanks, cleanup done. |
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>
2b05af4 to
6c304f7
Compare
|
Looks good to me. |
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
✅ Formalities