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 users to configure host, port and timeout for build in TCP adapters #56

Merged
merged 7 commits into from
Oct 27, 2024

Conversation

RemcoSmitsDev
Copy link
Owner

@RemcoSmitsDev RemcoSmitsDev commented Oct 26, 2024

This PR enhances user configurability for built-in adapters utilizing TCP transport by allowing customization of host, port, and timeout settings. Additionally, we've implemented a feature where, if no port is specified, the system will automatically attempt to locate an available port on the provided host.

TODO:

  • Update or remove the cached binaries functionality, as the host/port information is no longer stored within the DebugAdapterBinary

@RemcoSmitsDev RemcoSmitsDev changed the title Allow dynamic tcp ports Allow users to configure host, port and timeout Oct 26, 2024
@RemcoSmitsDev RemcoSmitsDev changed the title Allow users to configure host, port and timeout Allow users to configure host, port and timeout for build in TCP adapters Oct 26, 2024
RemcoSmitsDev and others added 3 commits October 27, 2024 11:56
…ne adapters

This was an issue, because we stored the port arg inside the DebugAdapterBinary which was cached so could not change anymore.

Co-authored-by: Anthony Eid <hello@anthonyeid.me>
Co-Authored-By: Anthony Eid <56899983+Anthony-Eid@users.noreply.github.com>
@RemcoSmitsDev RemcoSmitsDev marked this pull request as ready for review October 27, 2024 11:15
@RemcoSmitsDev RemcoSmitsDev merged commit f7eb521 into debugger Oct 27, 2024
2 of 6 checks passed
@RemcoSmitsDev RemcoSmitsDev deleted the allow-dynamic-tcp-ports branch October 27, 2024 11:16
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