Support TCP and UDP proxy #2078
Conversation
|
@buchdag Are there any clear requirements (e.g. tests) which prevent this PR from landing? Or is it that the benefit of this feature is simply not clear and thus polishing the PR does not make sense? I could do whatever is necessary to get this feature merged. Thanks a lot! |
|
@p12tic a test would be nice but the PR seems rather straightforward to me, the main issue is rather that it conflicts with the current code ... and that I completely failed to notice it. I'll try to fix the merge conflicts this week. |
I can create a separate PR with merge conflicts fixed (the authorship of the code remains with KagurazakaNyaa of course). |
|
No need to, I just rebased the branch and fixed the conflict. Could you review the PR and check that I did not botch anything ? |
|
Thanks. I will add a test in a subsequent PR. |
Add including to the root of nginx.conf to facilitate adding configuration outside the http module.