-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Env: Add XRAY_LOCATION_CERT
variable
#4536
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
Conversation
done. |
done, all you had to do was say rename all "certificate" to "cert" :) |
XRAY_LOCATION_CERT
variable
#4531 (comment) Fixes #4531 --------- Co-authored-by: RPRX <63339210+RPRX@users.noreply.github.com>
|
直接改readdfile方法去exec folder读一遍没有再fallback去asset目录(默认二进制位置)读一遍就行了 以后类似的东西直接调这个方法还能复用 这还单独搓个出来 |
we can add "readFileFromExec" function:
But it is not useful for our needs, because we should first check if the file-path is absolute then check the environment-variable and then check the exec folder. Or maybe I didn't understand what you meant, anyway, you can make any changes you want. |
我主要是觉得加个 反正我加这些东西都是从 |
比如以后可能再加个 好像暂时没其它涉及到 file 的配置项了 话说 |
@Fangliding 文档需要更正 |
This reverts commit 2d3210e.
XTLS#4531 (comment) Fixes XTLS#4531 --------- Co-authored-by: RPRX <63339210+RPRX@users.noreply.github.com> (cherry picked from commit 2d3210e)
This reverts commit 75723f9.
Wrongly, the source of relative path for certificate files is "command-run" folder: #4531
also, it is not elegant that we put all certificate files in exec folder, so we need environment-variable for certificate files.
therefore, gui/panel can create a folder for certificate files and set certificate-environment-variable to that folder.
also, v2rayNG only has one folder for asset files, so in v2rayNG we can set
geo-environment-variable=certificate-environment-variable and then we can also add certificate files.
///
logic: