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

Handle bi-directional communication between server and client #32

Closed
Tracked by #342 ...
mostafa opened this issue Nov 20, 2022 · 1 comment · Fixed by #245
Closed
Tracked by #342 ...

Handle bi-directional communication between server and client #32

mostafa opened this issue Nov 20, 2022 · 1 comment · Fixed by #245
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@mostafa
Copy link
Member

mostafa commented Nov 20, 2022

I suppose this can be done using Go channels.

gatewayd/network/proxy.go

Lines 123 to 127 in da83204

// TODO: Handle bi-directional traffic
// Currently the passthrough is a one-way street from the client to the server, that is,
// the client can send data to the server and receive the response back, but the server
// cannot take initiative and send data to the client. So, there should be another event-loop
// that listens for data from the server and sends it to the client

Update:
I've done some initial work in this branch, but most of this is related to server to client communication. Examples of which is NOTIFY command in PostgreSQL, which is not part of the SQL standard.

I'll close this ticket until there's an actual need for this.

@mostafa mostafa added the enhancement New feature or request label Nov 20, 2022
@mostafa mostafa added this to the v0.0.x milestone Nov 20, 2022
@mostafa mostafa self-assigned this Nov 20, 2022
@mostafa mostafa modified the milestones: v0.0.x, v0.1.x Dec 13, 2022
@mostafa mostafa moved this from 🆕 New to 🏗 In progress in GatewayD Core Public Roadmap Dec 14, 2022
@mostafa mostafa closed this as completed Dec 14, 2022
@mostafa mostafa moved this from 🏗 In progress to ✅ Done in GatewayD Core Public Roadmap Dec 14, 2022
@mostafa mostafa moved this from ✅ Done to 🏗 In progress in GatewayD Core Public Roadmap May 19, 2023
@mostafa mostafa modified the milestones: v0.1.x, v0.6.x May 19, 2023
@mostafa mostafa reopened this May 19, 2023
@github-project-automation github-project-automation bot moved this from 🏗 In progress to 📋 Backlog in GatewayD Core Public Roadmap May 19, 2023
@mostafa mostafa moved this from 📋 Backlog to 🏗 In progress in GatewayD Core Public Roadmap May 19, 2023
@mostafa mostafa linked a pull request May 20, 2023 that will close this issue
@mostafa mostafa moved this from 🏗 In progress to ✅ Done in GatewayD Core Public Roadmap May 20, 2023
@mostafa mostafa moved this from 🎉 Done to 🚧 In progress in GatewayD Core Public Roadmap Sep 29, 2023
@mostafa mostafa reopened this Sep 29, 2023
@github-project-automation github-project-automation bot moved this from 🚧 In progress to 📋 Backlog in GatewayD Core Public Roadmap Sep 29, 2023
@mostafa mostafa moved this from 📋 Backlog to 🚧 In progress in GatewayD Core Public Roadmap Sep 29, 2023
@mostafa mostafa removed this from the v0.6.x milestone Sep 29, 2023
@mostafa mostafa added this to the v0.8.x milestone Sep 29, 2023
@mostafa mostafa mentioned this issue Sep 30, 2023
4 tasks
@mostafa mostafa moved this from 🚧 In progress to 📋 Backlog in GatewayD Core Public Roadmap Sep 30, 2023
@mostafa
Copy link
Member Author

mostafa commented Oct 17, 2023

Fixed in #344.

@mostafa mostafa closed this as completed Oct 17, 2023
@mostafa mostafa moved this from 📋 Backlog to 👀 In review in GatewayD Core Public Roadmap Oct 17, 2023
@mostafa mostafa moved this from 👀 In review to 🔀 Merged in GatewayD Core Public Roadmap Oct 17, 2023
@mostafa mostafa moved this from 🔀 Merged to 🎉 Done in GatewayD Core Public Roadmap Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging a pull request may close this issue.

1 participant