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

ttyd crashes on windows #1292

Closed
tech128 opened this issue Jan 28, 2024 · 5 comments
Closed

ttyd crashes on windows #1292

tech128 opened this issue Jan 28, 2024 · 5 comments
Labels

Comments

@tech128
Copy link

tech128 commented Jan 28, 2024

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

  1. run ttyd.win32.exe cmd
  2. open browser and enter in the ip address:7681
  3. ttyd exits with either "== CreateProcessW failed with error 267: The directory name is invalid." or just nothing at all

Expected behavior
Expected to see a command prompt in the browser when I connect to the port.

Screenshots
N / A

Environment:

  • OS: Windows 10 and Windows 11
  • Browser: Edge 121.0.2277.83 (Official build) (64-bit)

Additional context
I traced it with process monitor and see it tries to reference some weird directory with chinese characters for some odd reason. I created this directory but it still does not fix the crash. Something is causing the crash right when it tries to run the command.

@tech128 tech128 added the bug label Jan 28, 2024
@tsl0922 tsl0922 added question and removed bug labels Jan 28, 2024
@tsl0922
Copy link
Owner

tsl0922 commented Jan 28, 2024

It has nothing to do with ttyd, you may have other file named as cmd, trying using full path like:

ttyd.win32.exe C:\Windows\System32\cmd.exe

@tsl0922 tsl0922 closed this as completed Jan 28, 2024
@tech128
Copy link
Author

tech128 commented Feb 4, 2024

Nope, still the same issue. I should have listed that in the report, but I figured you would look into it.

Just fyi, I'm not a noob who doesn't know if something is a bug or not.

If I post a report, trust me, it's a bug.

I go to great lengths to make sure its not really something I've done wrong somehow.

@tsl0922
Copy link
Owner

tsl0922 commented Feb 5, 2024

The error you encountered is returned from windows API, you may google it yourself.

@tech128
Copy link
Author

tech128 commented Feb 7, 2024

Well I did more than that, I did a lot of testing against the current and older versions and found that version 1.7.3 works fine on both. This report shows my exact issue.

#1207

I would have found it if it had a better title.

Thank you for writing this software, I use it as a quick form of remote access and also works well when the customer is still using his machine.

@dna2github
Copy link

dna2github commented Jul 11, 2024

i met the same problem @tech128
and i tried to set cwd like -w c:\ then it works ... therefore it is not a bug.
what i can suggest is that we may add a default cwd if not specified in source code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants