Skip to content

Implement the pass through - #172

Closed
jerryyummy wants to merge 10 commits into
mushorg:mainfrom
jerryyummy:main
Closed

Implement the pass through#172
jerryyummy wants to merge 10 commits into
mushorg:mainfrom
jerryyummy:main

Conversation

@jerryyummy

@jerryyummy jerryyummy commented Feb 6, 2025

Copy link
Copy Markdown

fix #161 implement passthrough so when you try to access a specific port, it will not set a handler for this port, and it just pass traffic to the target server not glutton

@jerryyummy

Copy link
Copy Markdown
Author

@glaslos could you please have a check?

Comment thread rules/rules.go Outdated
Comment thread config/rules.yaml Outdated
Comment thread glutton.go Outdated
}

md, err := g.connTable.RegisterConn(conn, rule)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why?

Comment thread config/rules.yaml
target: memcache
- match: tcp dst port 8080
type: pass_through
target: pass_through

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can use a host:port as a target here

Comment thread protocols/protocols.go Outdated
import (
"bytes"
"context"
"github.com/mushorg/glutton/rules"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please use go fmt

Comment on lines +41 to +44
go func() {
_, err := io.Copy(targetConn, conn)
errChan <- err
}()

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When is this go routine closed?

@glaslos

glaslos commented Mar 21, 2025

Copy link
Copy Markdown
Member

Please resolve the merge conflicts.

@glaslos

glaslos commented May 20, 2025

Copy link
Copy Markdown
Member

@jerryyummy are you still planning to work on this issue?

@jerryyummy

Copy link
Copy Markdown
Author

@jerryyummy are you still planning to work on this issue?

sorry about that, i can not solve it right now, but i will solve it if i have the time

@jerryyummy jerryyummy closed this by deleting the head repository Sep 25, 2025
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.

Implement a pass-through configuration

2 participants