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

allow clients to connect over unix domain sockets #166

Closed

Conversation

tdyas-stripe
Copy link

Allow debugger clients to connect to the in-process debug server over a UNIX domain socket instead of a local TCP socket. My company has been running a variant of this PR (against a 0.6.x release) internally since last September.

  • start_server takes an additional socket_path argument. An error is raised if the host parameter is also provided at the same time. I'd love suggestions on how to improve the API.

  • The PR refactors the tests enough to allow testing both the TCP socket and UNIX domain socket variants with the same test code.

@tdyas-stripe
Copy link
Author

Looks like CI builds failed for Ruby v1.8.7 and v1.9.3. Any suggestions on how to modify the API to work on those earlier versions?

@tdyas tdyas mentioned this pull request Dec 5, 2019
@tdyas
Copy link
Contributor

tdyas commented Dec 8, 2019

This should be closed in favor of #183.

@ViugiNick ViugiNick closed this Dec 11, 2019
@alir2t2
Copy link

alir2t2 commented Jun 25, 2022

To connect client using a port I used this configuration shown below:

Launch.json

{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "rdebug-ide",
            "type": "Ruby",
            "request": "attach",
            "remoteHost": "127.0.0.1",
            "remotePort": "1234",
            "remoteWorkspaceRoot": "${workspaceRoot}",
            "cwd": "${workspaceRoot}"
        }
    ]
}

rdebug-ide started with the following command:
rdebug-ide --port 1234 -- rails/server s

If I use unix socket instead of port, how do I connect my client to the unix socket?

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.

4 participants