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

App start fails saying the RDP port is not open #270

Open
tristanRW opened this issue Sep 23, 2024 · 1 comment
Open

App start fails saying the RDP port is not open #270

tristanRW opened this issue Sep 23, 2024 · 1 comment
Labels
bug Something isn't working good first issue Good for newcomers legacy Using legacy winapps

Comments

@tristanRW
Copy link
Contributor

The following command: timeout 5 nc -z "$RDP_IP" "$RDP_PORT" &>/dev/null; makes app startup fail, because it does not hear back over the rdp port, even though the VM is up and running RDP-server on said port.

This is also why setup.sh fails with Error 13 in some instances even though the VM is reachable through RDP. See #260. A fix should remedy both issues.

Maybe we can find an alternative to this port-check over netcat? Alternatively we could leave out the port-check entirely and just report the connection error if the rdp connection-attempt fails one step later in the connection-process.

@tristanRW
Copy link
Contributor Author

I propose removing this netcat-port-check.

If the rdp-command fails, we can still print its error, or catch it and report the connection not being possible. Checking with netcat for the reason rdp failed is also a possibility.

In both cases the connection via rdp is at least attempted before the user is told it is "impossible".
The current behaviour is especially misleading when connecting is possible, and works fine on re-run (of whatever app tried to connect).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers legacy Using legacy winapps
Projects
None yet
Development

No branches or pull requests

2 participants