Skip to content

remove type settings#866

Closed
LjhAUMEM wants to merge 1 commit into
XTLS:mainfrom
LjhAUMEM:fm
Closed

remove type settings#866
LjhAUMEM wants to merge 1 commit into
XTLS:mainfrom
LjhAUMEM:fm

Conversation

@LjhAUMEM

Copy link
Copy Markdown
Contributor

只看右边就够了吧

@Fangliding

Copy link
Copy Markdown
Member

真会有人认不出来的 而且这里和别的地方不一致 别的地方标题也不代表字段 别的地方都是 XxxxObject 就不是字段名称

@Fangliding Fangliding closed this Jun 15, 2026
@LjhAUMEM

Copy link
Copy Markdown
Contributor Author

唉,风格问题,不过我原来的计划是有时间再把子标题下的内容做成可展开的,这样打开这页就是可做 type 的子标题 + 紧跟未展开的 settings,和现在 type 列一堆效果应该差不多,而且以后修改就不用管上面的 type。既然关了我就不管了,谁想写谁写吧,不然别人还以为我是 m 喜欢被关 pr

(PS: 不是两次艾特我也不会来这 XTLS/Xray-core#6193 (comment) XTLS/Xray-core#6245 (comment))

别的地方标题也不代表字段 别的地方都是 XxxxObject 就不是字段名称

这个别的地方关 fm 什么事,fm 的标题能做 type 不就行了

@Fangliding

Copy link
Copy Markdown
Member

改上又改下这种事不少 比如以前的sockopt最上面一个 streamSettings 下面 sockopt 又列一条 然后本身的解释又一条 顺序还可能倒不一样 以前就经常弄错也有不少pr补或者被自查找到 填typo一样一个pr就增删个几行 毕竟不是程序写个注册入口其他地方自动搞好 我是习惯了觉得不是啥问题 现在这种小 typo 我都丢给 AI 查了也不费多少事

@RPRX

RPRX commented Jun 17, 2026

Copy link
Copy Markdown
Member

@LjhAUMEM 直接移除的话确实可能有人不能直接 get 到 type 该写的是下面的小标题,还是更新列表吧

settings 那个列表可以删了,现在这意思也不对

另外“数组第一个为最外层伪装”应该是“数组第一个为最内层伪装”

@RPRX RPRX reopened this Jun 17, 2026
@LjhAUMEM

Copy link
Copy Markdown
Contributor Author

另外“数组第一个为最外层伪装”应该是“数组第一个为最内层伪装”

最外层确实是第一个,最后 write 最先 read

@Fangliding

Copy link
Copy Markdown
Member

不用了 我看到之后 mkcp-legcay 那里的问题已经在弄一个更新了 多写一层就是 这是示例
image

@Fangliding Fangliding closed this Jun 17, 2026
@RPRX

RPRX commented Jun 17, 2026

Copy link
Copy Markdown
Member

最外层确实是第一个,最后 write 最先 read

这有点反人类直觉了啊,也不符合 ray 的架构,按层层叠加来说最外层应该是最后一个,你改下 core 的代码吧 @LjhAUMEM

@LjhAUMEM

Copy link
Copy Markdown
Contributor Author

刚刚 fp 了我说怎么没提示,原来又 closed 了

@LjhAUMEM 直接移除的话确实可能有人不能直接 get 到 type 该写的是下面的小标题,还是更新列表吧

settings 那个列表可以删了,现在这意思也不对

这个文档我之后还是放手?

@Fangliding type 除了少 realm 还有 gecko

@LjhAUMEM

Copy link
Copy Markdown
Contributor Author

这有点反人类直觉了啊,也不符合 ray 的架构,按层层叠加来说最外层应该是最后一个,你改下 core 的代码吧 @LjhAUMEM

那除了 infra 还要改 header manager,变来变去不好吧

@Fangliding

Copy link
Copy Markdown
Member

你看现在这样没问题就没问题吧
type列表已经按你之前那样全部删掉了 平铺到下面 我之前的意思是就这么草率删了让看文档的人凭悟猜type应该按标题填不好 realm就不少了(它的介绍本来就存在)gecko 是完全不存在再额外写一条就是

@RPRX

RPRX commented Jun 17, 2026

Copy link
Copy Markdown
Member

@LjhAUMEM 我本来设想的就是符合人类直觉的“层层叠加”,趁着现在基本上也就只用了一个 mask 还是快点纠正过来吧

@LjhAUMEM

Copy link
Copy Markdown
Contributor Author

@Fangliding settings 应该是 Object
image
风格这方面还是按照主维护者的意思,我其实不是很想弄文档,但是让我来弄就是怎么方便怎么来

@LjhAUMEM

Copy link
Copy Markdown
Contributor Author

@LjhAUMEM 我本来设想的就是符合人类直觉的“层层叠加”,趁着现在基本上也就只用了一个 mask 还是快点纠正过来吧

好吧,应该可以只修改 finalmask.go 一个文件,我看看

@RPRX

RPRX commented Jun 17, 2026

Copy link
Copy Markdown
Member

@Fangliding string 那处,以及改成“数组第一个为最内层伪装”,fp 一下

@Fangliding

Fangliding commented Jun 17, 2026

Copy link
Copy Markdown
Member
image 这个packetsize好像不符合空字符串/0=不存在=默认的规则 不存在和填空串是两个行为

@Fangliding

Fangliding commented Jun 17, 2026

Copy link
Copy Markdown
Member

先说好是留这样不该 不存在→不开 存在→默认512~120 还是改成看 To=0(填0或者空字符串都会变成 To=0) 则不开 要不又要改一道了

@LjhAUMEM

Copy link
Copy Markdown
Contributor Author

现在填空是会应用默认的 512-1200

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.

3 participants