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

Update README.md #2459

Merged
merged 1 commit into from
Aug 20, 2023
Merged

Update README.md #2459

merged 1 commit into from
Aug 20, 2023

Conversation

hel1o-wor1d
Copy link
Contributor

A better docker image of xray

A better docker image of xray
@yuhan6665 yuhan6665 merged commit 75c99e2 into XTLS:main Aug 20, 2023
@yuhan6665
Copy link
Member

thanks!

@RPRX
Copy link
Member

RPRX commented Apr 26, 2024

A better docker image of xray

随便一个人跑过来说我有个 better docker image,better 在哪?同质化严重,且如 #2837 所言我们不确定能否信任他,没什么资料

这个 docker image 早就没更新了,幸好不是有心下毒不然已经中招很久了,所以下次这种奇奇怪怪的 PR 不要合了,等下我删掉

RPRX added a commit that referenced this pull request Apr 26, 2024
@Fangliding
Copy link
Member

Fangliding commented Apr 26, 2024

@RPRX
在尝试修文档的时候就注意到这个docker有问题了 不过后面睡一觉起来把这事忘了()
顺便提一嘴 readme 里那三个 hiddify系列里 manager似乎是假开源/部分开源 而且那个组织下面有一个singbox客户端甚至还是违反开源许可证的()
许可证的事暂时不提 毕竟不在里面 有没有必要要求readme里的项目完全开源?(那个manager甚至在自己的readme写了 recommend by xray)

@yuhan6665
Copy link
Member

我觉得不开源也可以 毕竟有ios 把两类分开写好了

@Fangliding
Copy link
Member

我觉得不开源也可以 毕竟有ios 把两类分开写好了

苹果软件不能开嘛 我记得SFI就是开的呀(

@yuhan6665
Copy link
Member

理论上可以开 实际上没人愿意开

RPRX added a commit that referenced this pull request Apr 26, 2024
@RPRX
Copy link
Member

RPRX commented Apr 26, 2024

你们排查一下,源代码可见的附上括号 链接,源代码非全部可见的都标个括号 closed-source 吧,比如知名 GUI 客户端 X-flutter

我们关心的是能否审计代码,所以看源协议也行,就看源代码可不可见

Recommended by Xray

麻烦修改为 listed by Xray @hiddify-com

mmmray added a commit to mmmray/Hiddify-Manager that referenced this pull request Jun 30, 2024
@hiddify-com
Copy link
Contributor

Dear @RPRX
Thank you for your comment. We have changed the word as you requested.

Please note that we are fully open-source and all our services are fully automatically built by GitHub action and release to all stores without any hands :-).

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.

None yet

5 participants