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

server, client: connectRPCObject for web api clients #1166

Merged
merged 11 commits into from
Nov 8, 2023

Conversation

bjia56
Copy link
Collaborator

@bjia56 bjia56 commented Nov 6, 2023

Uses a new engine.io endpoint to serve as the transport for connectRPCObject requests coming from the web client.

packages/client/src/index.ts Outdated Show resolved Hide resolved
packages/client/src/index.ts Outdated Show resolved Hide resolved
packages/client/src/index.ts Outdated Show resolved Hide resolved
@koush
Copy link
Owner

koush commented Nov 7, 2023

uhh disregard that change, didn't realize I was working in your branch.

@koush koush force-pushed the connect-object-web branch from 4292254 to a50dda8 Compare November 7, 2023 17:34
@koush
Copy link
Owner

koush commented Nov 7, 2023

ok backed it out with a force push, pretty sure this was the state before I clobbered your stuff.

packages/client/src/index.ts Outdated Show resolved Hide resolved
server/src/runtime.ts Outdated Show resolved Hide resolved
packages/client/src/index.ts Outdated Show resolved Hide resolved
* Handle incoming connections that will be
* proxied to a connectRPCObject socket.
*/
export function setupConnectRPCObjectProxy(clusterSecret: string, port: number, connection: Socket & IOSocket) {
Copy link
Owner

Choose a reason for hiding this comment

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

I think you mean Socket | IOSocket

@koush koush merged commit e49f26b into koush:main Nov 8, 2023
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.

2 participants