Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Proxy protocol moves tolog transport #156

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

bazsi
Copy link
Member

@bazsi bazsi commented Jun 9, 2024

This is now a complete LogTransport based implementation of the HAProxy protocol so it applies cleanly to any kind of LogProto implementation.

@bazsi bazsi force-pushed the proxy-protocol-moves-tolog-transport branch from 4ba6ca2 to a496b05 Compare September 15, 2024 17:50
@bazsi bazsi force-pushed the proxy-protocol-moves-tolog-transport branch 4 times, most recently from 5588b6a to 053c429 Compare September 18, 2024 08:04
@bazsi bazsi force-pushed the proxy-protocol-moves-tolog-transport branch from 053c429 to 1e92e02 Compare September 18, 2024 10:19
bazsi and others added 15 commits September 20, 2024 23:48
Signed-off-by: Balazs Scheidler <balazs.scheidler@axoflow.com>
…transport

Signed-off-by: Balazs Scheidler <balazs.scheidler@axoflow.com>
…port()

Signed-off-by: Balazs Scheidler <balazs.scheidler@axoflow.com>
Signed-off-by: Balazs Scheidler <balazs.scheidler@axoflow.com>
Signed-off-by: Balazs Scheidler <balazs.scheidler@axoflow.com>
These are interdependent interfaces and LogTransportStack will be the
sole user of LogTransportFactory.

Signed-off-by: Balazs Scheidler <balazs.scheidler@axoflow.com>
Instead of relying on the factory to initialize name, initialize it
through the normal constructor path.

Signed-off-by: Balazs Scheidler <balazs.scheidler@axoflow.com>
…a stack of LogTransports

Instead of trying to reuse the LogTransport interface and hide cases where
we need multiple swappable LogTransports, let's introduce a
LogTransportStack, that explicitly tracks all such LogTransports, takes
care of their lifecycle and allows them to be swapped during a connection.

To be used for HA Proxy protocol support.

Signed-off-by: Balazs Scheidler <balazs.scheidler@axoflow.com>
Signed-off-by: Balazs Scheidler <balazs.scheidler@axoflow.com>
This patch is based on the LogTransport implementation in
github.com/syslog-ng/syslog-ng, which in turn is based on my original
implementation of proxy support as a LogProto class.

Signed-off-by: Alex Becker <beckeralex@protonmail.com>
Signed-off-by: Hofi <hofione@gmail.com>
Signed-off-by: Balazs Scheidler <balazs.scheidler@axoflow.com>
Signed-off-by: Balazs Scheidler <balazs.scheidler@axoflow.com>
Signed-off-by: Alex Becker <beckeralex@protonmail.com>
Signed-off-by: Hofi <hofione@gmail.com>
Signed-off-by: Balazs Scheidler <balazs.scheidler@axoflow.com>
…roxied' loggen parameter

Signed-off-by: Hofi <hofione@gmail.com>
…e loggen with proper `proxied` switches

Signed-off-by: Hofi <hofione@gmail.com>
@bazsi bazsi force-pushed the proxy-protocol-moves-tolog-transport branch from 1e92e02 to a8c2436 Compare September 20, 2024 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants