Skip to content

add: sse, streamable support passing the --host parameter - #185

Merged
hustcc merged 12 commits into
antvis:mainfrom
Unexpectedlyc:main
Oct 9, 2025
Merged

add: sse, streamable support passing the --host parameter#185
hustcc merged 12 commits into
antvis:mainfrom
Unexpectedlyc:main

Conversation

@Unexpectedlyc

@Unexpectedlyc Unexpectedlyc commented Sep 2, 2025

Copy link
Copy Markdown
Contributor

add: sse, streamable support passing the --host parameter

Comment thread src/index.ts Outdated
@hustcc

hustcc commented Sep 3, 2025

Copy link
Copy Markdown
Member

-h 是不是应该 bool 值,如果有那就可以内网通过 ip 被访问到?

@Unexpectedlyc

Copy link
Copy Markdown
Contributor Author

-h 是不是应该 bool 值,如果有那就可以内网通过 ip 被访问到?

不是,是string,可以根据自己的内网ip修改,比如--host 192.168.101.10

@hustcc

hustcc commented Sep 3, 2025

Copy link
Copy Markdown
Member

不是,是string,可以根据自己的内网ip修改,比如--host 192.168.101.10

我使用过的 cli 工具,看起来是 host 开启的时候,自动获取本机 ip。

@Unexpectedlyc

Copy link
Copy Markdown
Contributor Author

不是,是string,可以根据自己的内网ip修改,比如--host 192.168.101.10

我使用过的 cli 工具,看起来是 host 开启的时候,自动获取本机 ip。

没有吧 ,我用的很多都是自己指定ip的,而且如果用户有多个网卡的话,取哪个ip也是个问题吧

@hustcc

hustcc commented Sep 4, 2025

Copy link
Copy Markdown
Member

@Unexpectedlyc 可以参考 webpack dev server 或者 vite

建议可以按照 vite 的,更简单一些。

  • 默认 host 为 localhost
  • --host 开启后,默认 host 为 0.0.0.0
  • -- host 开启后,可以指定 ip

@Unexpectedlyc

Copy link
Copy Markdown
Contributor Author

@Unexpectedlyc 可以参考 webpack dev server 或者 vite

建议可以按照 vite 的,更简单一些。

  • 默认 host 为 localhost
  • --host 开启后,默认 host 为 0.0.0.0
  • -- host 开启后,可以指定 ip

@hustcc 目前第一点跟第三点是满足的,第二点我感觉没有必要吧?

@hustcc

hustcc commented Sep 4, 2025

Copy link
Copy Markdown
Member

@hustcc 目前第一点跟第三点是满足的,第二点我感觉没有必要吧?

感觉挺有必要的啊,不然用户 --host 还需要自己输入自己本机的内网 ip,这个本身就是一个门槛吧!

@hustcc

hustcc commented Sep 5, 2025

Copy link
Copy Markdown
Member
image

vite 的开发服务,也是这样的!

@Unexpectedlyc

Copy link
Copy Markdown
Contributor Author
image vite 的开发服务,也是这样的!

好的,我会参考vite继续改进的

- Change host argument type from string to boolean
- Implement custom logic to extract host value from command line tokens
- Set default host value to localhost
- If --host flag is set without a value, default to 0.0.0.0
- Update parsed values object to include the determined host value
@Unexpectedlyc

Unexpectedlyc commented Sep 6, 2025

Copy link
Copy Markdown
Contributor Author

@Unexpectedlyc 可以参考 webpack dev server 或者 vite

建议可以按照 vite 的,更简单一些。

  • 默认 host 为 localhost
  • --host 开启后,默认 host 为 0.0.0.0
  • -- host 开启后,可以指定 ip

@hustcc 已经完成了,review一下,目前基于node原生的parseargs只能用这种方式实现一下了

Comment thread src/index.ts
Comment thread src/index.ts
Comment thread src/index.ts
Comment thread src/index.ts Outdated
Comment thread src/server.ts
@FailedNamed

Copy link
Copy Markdown

hello,请问这个commit大概啥时候可以合并且打包到npm包里,现在安装的还不支持这个host参数

@hustcc
hustcc merged commit f19d8f0 into antvis:main Oct 9, 2025
1 check passed
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.

命令行运行,为什么没有--host参数呀...

3 participants