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

Cannot run on windows #6

Open
drunkenQCat opened this issue Aug 25, 2024 · 11 comments
Open

Cannot run on windows #6

drunkenQCat opened this issue Aug 25, 2024 · 11 comments
Assignees

Comments

@drunkenQCat
Copy link

After run deno run -A jsr:@ryoppippi/curxy, the terminal feedback:

Listening on http://127.0.0.1:63263/
◐ Starting cloudflared tunnel to http://127.0.0.1:63263                                                 10:44:19
error: Uncaught (in promise) TypeError: Windows only supports ctrl-c (SIGINT) and ctrl-break (SIGBREAK).
    at bindSignal (ext:runtime/40_signals.js:14:10)
    at Object.addSignalListener (ext:runtime/40_signals.js:54:19)
    at Process.on (node:process:355:12)
    at Process.once (ext:deno_node/_events.mjs:533:8)
    at startTunnel (file:///C:/Users/user/AppData/Local/deno/npm/registry.npmmirror.com/untun/0.1.3/dist/index.mjs:36:13)
    at async Promise.all (index 1)
    at async https://jsr.io/@ryoppippi/curxy/0.1.5/main.ts:73:3
    ```
@ryoppippi
Copy link
Owner

I don't use windows so prs are welcome

@ekinertac
Copy link

just sent a PR

@ryoppippi
Copy link
Owner

ryoppippi commented Sep 4, 2024

@ekinertac hasn't updated his PR for a while.
If anyone want to contribute it, I'm happy about it

@ekinertac
Copy link

couldn't find time to update the PR, all contributions are welcome

@ryoppippi
Copy link
Owner

@ekinertac thank you for your initial work!

@hgoona
Copy link

hgoona commented Sep 11, 2024

Hi @ryoppippi ! Is this now able to be used on Windows or is it still not setup?🤔 I'm new to this kind of thing and just came across your package..

@ryoppippi
Copy link
Owner

@hgoona
Hi!
I don't have any Windows machine, so idk it works on Windows.

However, I added a flag to disable cloudflared, so you can run it on windows by disabling it because this error is about cloudflared.

@hgoona
Copy link

hgoona commented Sep 11, 2024

@ryoppippi Oh - how do make that change?
Im trying it now and I see this if i run deno run -A jsr:@ryoppippi/curxy
image

@ryoppippi
Copy link
Owner

I already added cloudflared flag, so you can just run deno run -A jsr:@ryoppippi/curxy --cloudflared=false
But in this case, the localhost will not be public and you need to find some other way to make it public(maybe using ngrok is another option?)

@ryoppippi
Copy link
Owner

ryoppippi commented Sep 12, 2024

Also, this issue is untun's issue, so please someone to make an issue to untun.
If you guys have any alternative, send me a PR!
Contributions are always welcome

@ryoppippi
Copy link
Owner

untun is widly used like nuxt or slidev, so idk why it does not work in deno.
If it doesn't work on deno but on node, maybe we re-implement it to use node/bun instead of deno

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 a pull request may close this issue.

4 participants