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.

Sign up for GitHub

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

Jump to bottom

v2ray.service upgrade #2091

Merged
merged 3 commits into from
Feb 18, 2020
Merged

v2ray.service upgrade #2091

merged 3 commits into from
Feb 18, 2020

Conversation

unknowndevQwQ
Copy link
Contributor

做了一点微小的工作，例如：
去掉了在非 forking 类型中几乎无意义的 PIDFile （因为 systemd 不会写入）
把不推荐使用的 Wants=network.target 换成 Wants=network-online.target nss-lookup.target

@unknowndevQwQ
v2ray.service upgrade
48c712d 
做了一点微小的工作，例如：
去掉了在非 `forking` 类型中几乎无意义的 `PIDFile` （[因为 systemd 不会写入](https://www.freedesktop.org/software/systemd/man/systemd.service.html#PIDFile=)）
把[不推荐使用](https://www.freedesktop.org/wiki/Software/systemd/NetworkTarget/)的 `Wants=network.target` 换成 `Wants=network-online.target nss-lookup.target`
@unknowndevQwQ
Copy link
Contributor Author

需要什么回复吗·

vcptr
vcptr suggested changes Dec 8, 2019
View reviewed changes
@xiaokangwang
Copy link
Contributor

V2Ray为什么需要依赖nss呢？是否可以解释一下具体的原因。

@xiaokangwang
Copy link
Contributor

network-online在有多个网卡但是只使用了其中一个的系统上会不会出现需要等待到超时的情况？

@xiaokangwang
Copy link
Contributor

V2Ray在启动后并不会主动进行网络通信。除非监听地址是某个interface的地址，否则这么改并没有明显的好处。

@unknowndevQwQ
Copy link
Contributor Author

unknowndevQwQ commented Dec 9, 2019 •
edited
Loading

V2Ray为什么需要依赖nss呢？是否可以解释一下具体的原因。

这个的话，主要是为了保证启动时域名可以正常进行解析

@unknowndevQwQ
Copy link
Contributor Author

unknowndevQwQ commented Dec 9, 2019 •
edited
Loading

network-online在有多个网卡但是只使用了其中一个的系统上会不会出现需要等待到超时的情况？

这个得看 network-online 对应的服务是怎么测的，大多只要是能联网就没问题

@xiaokangwang
Copy link
Contributor

V2Ray为什么需要依赖nss呢？是否可以解释一下具体的原因。

这个的话，主要是为了保证启动时域名可以正常进行解析

什么情况下需要在启动的时候解析域名呢？

@unknowndevQwQ
Copy link
Contributor Author

V2Ray为什么需要依赖nss呢？是否可以解释一下具体的原因。

这个的话，主要是为了保证启动时域名可以正常进行解析

什么情况下需要在启动的时候解析域名呢？

包括但不限于中转
或者在网关把流量扔进 v2ray 的时候

@unknowndevQwQ
Update v2ray.service
e92eddb 
default Type=exec -> Type=simple
@xiaokangwang
Copy link
Contributor

V2Ray为什么需要依赖nss呢？是否可以解释一下具体的原因。

这个的话，主要是为了保证启动时域名可以正常进行解析

什么情况下需要在启动的时候解析域名呢？

包括但不限于中转
或者在网关把流量扔进 v2ray 的时候

但是既然有流量进入的话，就说明网络已经起来了呀。我现在比较担心在实际流量已经开始来的情况下，V2还在等这个network-online的可能

@unknowndevQwQ
Copy link
Contributor Author

V2Ray为什么需要依赖nss呢？是否可以解释一下具体的原因。

这个的话，主要是为了保证启动时域名可以正常进行解析

什么情况下需要在启动的时候解析域名呢？

包括但不限于中转
或者在网关把流量扔进 v2ray 的时候

但是既然有流量进入的话，就说明网络已经起来了呀。我现在比较担心在实际流量已经开始来的情况下，V2还在等这个network-online的可能

已知的 NetworkManager-wait-online.service 与 systemd-wait-online.service 在网络可用时就会准备好的吧

顺便 Wants= 似乎并不是强制要求的依赖

@unknowndevQwQ
minor changes
a709ea5 
Weaken dependence on nss-lookup.target
Modify comments about #1011
@kslr kslr merged commit 175218b into v2ray:master Feb 18, 2020
@unknowndevQwQ
Copy link
Contributor Author

我过一会似乎还要提一个 pr
注释写错了，尴尬

xiocode pushed a commit to going/v2ray-core that referenced this pull request Jun 29, 2023
@chika0801
Replace Wings X with FoXray in README/GUI Clients (v2ray#2091)
51b2922
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kslr kslr kslr left review comments

@vcptr vcptr vcptr requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@unknowndevQwQ @xiaokangwang @kslr @vcptr