Skip to content

Update proto#187

Merged
RPRX merged 3 commits intomainfrom
unknown repository
Jan 21, 2021
Merged

Update proto#187
RPRX merged 3 commits intomainfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jan 19, 2021

No description provided.

@RPRX
Copy link
Copy Markdown
Member

RPRX commented Jan 21, 2021

go mod 新增的 indirect 那行可以去掉

@RPRX
Copy link
Copy Markdown
Member

RPRX commented Jan 21, 2021

go mod 新增的 indirect 那行可以去掉

可能是在当前项目中直接更新生成工具而引入的

另外我这边最新 protoc-gen-gofast 生成 VLESS 那个 .pb.go 好像大改了

@RPRX
Copy link
Copy Markdown
Member

RPRX commented Jan 21, 2021

go mod 新增的 indirect 那行可以去掉

可能是在当前项目中直接更新生成工具而引入的

另外我这边最新 protoc-gen-gofast 生成 VLESS 那个 .pb.go 好像大改了

破案了:不能在 Xray-core 项目路径下直接 install github.com/gogo/protobuf/protoc-gen-gofast,因为 go.sum 中有行 github.com/gogo/protobuf v1.1.1/go.mod h1:r8qH/GZQm5c6nD/R0oafs1akxWv10x8SbQlK7atdtwQ=,会导致安装到远古版本

我习惯新建一个临时文件夹和 go.mod 来手动安装/更新工具,然后直接 go run ./infra/vprotogen,这样也是没问题的
cd core + go generate 方式也不存在这个问题,因为 core/proto.go 中会始终安装/更新到最新版的 protoc-gen-gofast

顺便发现:v2fly/v2ray-core#366

@RPRX RPRX merged commit 7f5e34c into XTLS:main Jan 21, 2021
github-actions bot added a commit to sbily1988/Xray-core that referenced this pull request Jan 22, 2021
* https://github.com/XTLS/Xray-core:
  v1.2.3
  Changes from v2fly-core (XTLS#173)
  Fix fallbacks xver when original address is not TCP address (XTLS#182)
  Convert domain names to lowercase before matching (XTLS#195)
  Feature: Exclude some domains in sniffing destOverride (XTLS#151)
  Regenerate .pb.go files (XTLS#187)
it2konst pushed a commit to it2konst/gametunnel-core that referenced this pull request Mar 1, 2026
Co-authored-by: RPRX <63339210+rprx@users.noreply.github.com>
drovosek229 pushed a commit to drovosek229/Xray-core that referenced this pull request Mar 16, 2026
Co-authored-by: RPRX <63339210+rprx@users.noreply.github.com>
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.

1 participant