Skip to content

header-custom finalmask: Remove headerConnMode headerReadAddrAware interface#6193

Merged
RPRX merged 2 commits into
XTLS:mainfrom
LjhAUMEM:calc
May 28, 2026
Merged

header-custom finalmask: Remove headerConnMode headerReadAddrAware interface#6193
RPRX merged 2 commits into
XTLS:mainfrom
LjhAUMEM:calc

Conversation

@LjhAUMEM
Copy link
Copy Markdown
Collaborator

先把框架回退到 175502d 之前,唯一的影响就是 udp header 用不了 src_port_u16 dst_port_u16 src_ip4_u32 dst_ip4_u32 的变量,不影响 udp standalone,如果 @fatyzzz 认为 udp header 需要 addr,应当把 udp header 移除 header manager 而不是在 manager 改一堆只是为了 udp header

@LjhAUMEM
Copy link
Copy Markdown
Collaborator Author

现在才发现怎么他的测试都写到 ‎common/serial 去了

@fatyzzz
Copy link
Copy Markdown
Contributor

fatyzzz commented May 25, 2026

UDP standalone is enough for my current use case, so rollback is fine.

I still think addr-aware UDP headers may be useful later for more realistic protocol emulation, but moving that logic out of header manager sounds cleaner.

@LjhAUMEM
Copy link
Copy Markdown
Collaborator Author

可以让 udp header 作为独立的 conn 来使用 addr,因为近期准备重构 manager,所以不适合保留这些为一个 header 做的修改

@Meo597
Copy link
Copy Markdown
Collaborator

Meo597 commented May 25, 2026

这里的文档一直缺失,代码太长了人类估计也不会去翻,因此应该不会有人受到影响

@RPRX
Copy link
Copy Markdown
Member

RPRX commented May 26, 2026

话说我觉得 finalmask 文档里 UDP 那么多 header-* 但用的人又很少,有点刷屏了,要不把其它 header-* 都删掉换成模板吧

另外文档写错了,比如“一个数组,用以伪装核心发出的 TCP流量,数组第一个为最外层伪装。”写反了,细节少了个空格

@fatyzzz 的 programmable handshake 实在不行的话也可以单独弄个名字出来,以及 Hy2 Gecko 该加成 QUIC fragment?

@LjhAUMEM
Copy link
Copy Markdown
Collaborator Author

话说我觉得 finalmask 文档里 UDP 那么多 header-* 但用的人又很少,有点刷屏了,要不把其它 header-* 都删掉换成模板吧

header 我看下能怎么整合一下,也许来个 mkcp header

以及 Hy2 Gecko 该加成 QUIC fragment?

gecko 说白了就是给 salamander 伪装长度用的,跟 quic 有关系但不大

@RPRX
Copy link
Copy Markdown
Member

RPRX commented May 26, 2026

我的意思是那么多 header-* 真没啥人用还刷屏,不如删掉改成在文档中写出 example,即使需要 random 似乎已经支持了?

@RPRX RPRX changed the title remove headerConnMode headerReadAddrAware interface header-custom finalmask: Remove headerConnMode headerReadAddrAware interface May 28, 2026
@RPRX RPRX merged commit 1cd7d25 into XTLS:main May 28, 2026
39 checks passed
@MoRanYue
Copy link
Copy Markdown

header 我看下能怎么整合一下,也许来个 mkcp header

(这里认为,再次出现如此重大的破坏性变更并不好……)

@LjhAUMEM
Copy link
Copy Markdown
Collaborator Author

(这里认为,再次出现如此重大的破坏性变更并不好……)

调整还是有必要的,header实在太多了,可以帮忙想一下有没有更好的infra,现在暂定

"finalmask": {
  "udp": [
    {
      "type": "mkcp-legacy",
      "settings": {
        "type": "xxx", // aes128gcm original header
        "password": "",
        "headerID": 0,
        "domain": ""
      }
    }
  ]
}

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.

5 participants