mirror of
https://github.com/EasyTier/EasyTier.git
synced 2026-05-16 02:45:41 +00:00
Compare commits
15 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 253597ed02 | |||
| 6887d04233 | |||
| 05657db371 | |||
| 12a7b5a5c5 | |||
| 4eba9b07b6 | |||
| 1b48029bdc | |||
| 3542e944cb | |||
| 852d1c9e14 | |||
| 4958394469 | |||
| 41b6d65604 | |||
| aae30894dd | |||
| 81d169abfc | |||
| 9c6c210e89 | |||
| d1c6dcf754 | |||
| 97c8c4f55a |
@@ -11,7 +11,7 @@ on:
|
|||||||
image_tag:
|
image_tag:
|
||||||
description: 'Tag for this image build'
|
description: 'Tag for this image build'
|
||||||
type: string
|
type: string
|
||||||
default: 'v2.6.2'
|
default: 'v2.6.3'
|
||||||
required: true
|
required: true
|
||||||
mark_latest:
|
mark_latest:
|
||||||
description: 'Mark this image as latest'
|
description: 'Mark this image as latest'
|
||||||
|
|||||||
@@ -18,7 +18,7 @@ on:
|
|||||||
version:
|
version:
|
||||||
description: 'Version for this release'
|
description: 'Version for this release'
|
||||||
type: string
|
type: string
|
||||||
default: 'v2.6.2'
|
default: 'v2.6.3'
|
||||||
required: true
|
required: true
|
||||||
make_latest:
|
make_latest:
|
||||||
description: 'Mark this release as latest'
|
description: 'Mark this release as latest'
|
||||||
|
|||||||
Generated
+3
-3
@@ -2229,7 +2229,7 @@ checksum = "d0881ea181b1df73ff77ffaaf9c7544ecc11e82fba9b5f27b262a3c73a332555"
|
|||||||
|
|
||||||
[[package]]
|
[[package]]
|
||||||
name = "easytier"
|
name = "easytier"
|
||||||
version = "2.6.2"
|
version = "2.6.3"
|
||||||
dependencies = [
|
dependencies = [
|
||||||
"aes-gcm",
|
"aes-gcm",
|
||||||
"anyhow",
|
"anyhow",
|
||||||
@@ -2405,7 +2405,7 @@ dependencies = [
|
|||||||
|
|
||||||
[[package]]
|
[[package]]
|
||||||
name = "easytier-gui"
|
name = "easytier-gui"
|
||||||
version = "2.6.2"
|
version = "2.6.3"
|
||||||
dependencies = [
|
dependencies = [
|
||||||
"anyhow",
|
"anyhow",
|
||||||
"async-trait",
|
"async-trait",
|
||||||
@@ -2486,7 +2486,7 @@ dependencies = [
|
|||||||
|
|
||||||
[[package]]
|
[[package]]
|
||||||
name = "easytier-web"
|
name = "easytier-web"
|
||||||
version = "2.6.2"
|
version = "2.6.3"
|
||||||
dependencies = [
|
dependencies = [
|
||||||
"anyhow",
|
"anyhow",
|
||||||
"async-trait",
|
"async-trait",
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
id=easytier_magisk
|
id=easytier_magisk
|
||||||
name=EasyTier_Magisk
|
name=EasyTier_Magisk
|
||||||
version=v2.6.2
|
version=v2.6.3
|
||||||
versionCode=1
|
versionCode=1
|
||||||
author=EasyTier
|
author=EasyTier
|
||||||
description=easytier magisk module @EasyTier(https://github.com/EasyTier/EasyTier)
|
description=easytier magisk module @EasyTier(https://github.com/EasyTier/EasyTier)
|
||||||
|
|||||||
@@ -1,7 +1,7 @@
|
|||||||
{
|
{
|
||||||
"name": "easytier-gui",
|
"name": "easytier-gui",
|
||||||
"type": "module",
|
"type": "module",
|
||||||
"version": "2.6.2",
|
"version": "2.6.3",
|
||||||
"private": true,
|
"private": true,
|
||||||
"packageManager": "pnpm@9.12.1+sha512.e5a7e52a4183a02d5931057f7a0dbff9d5e9ce3161e33fa68ae392125b79282a8a8a470a51dfc8a0ed86221442eb2fb57019b0990ed24fab519bf0e1bc5ccfc4",
|
"packageManager": "pnpm@9.12.1+sha512.e5a7e52a4183a02d5931057f7a0dbff9d5e9ce3161e33fa68ae392125b79282a8a8a470a51dfc8a0ed86221442eb2fb57019b0990ed24fab519bf0e1bc5ccfc4",
|
||||||
"scripts": {
|
"scripts": {
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
[package]
|
[package]
|
||||||
name = "easytier-gui"
|
name = "easytier-gui"
|
||||||
version = "2.6.2"
|
version = "2.6.3"
|
||||||
description = "EasyTier GUI"
|
description = "EasyTier GUI"
|
||||||
authors = ["you"]
|
authors = ["you"]
|
||||||
edition.workspace = true
|
edition.workspace = true
|
||||||
|
|||||||
@@ -17,7 +17,7 @@
|
|||||||
"createUpdaterArtifacts": false
|
"createUpdaterArtifacts": false
|
||||||
},
|
},
|
||||||
"productName": "easytier-gui",
|
"productName": "easytier-gui",
|
||||||
"version": "2.6.2",
|
"version": "2.6.3",
|
||||||
"identifier": "com.kkrainbow.easytier",
|
"identifier": "com.kkrainbow.easytier",
|
||||||
"plugins": {
|
"plugins": {
|
||||||
"shell": {
|
"shell": {
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
[package]
|
[package]
|
||||||
name = "easytier-web"
|
name = "easytier-web"
|
||||||
version = "2.6.2"
|
version = "2.6.3"
|
||||||
edition.workspace = true
|
edition.workspace = true
|
||||||
description = "Config server for easytier. easytier-core gets config from this and web frontend use it as restful api server."
|
description = "Config server for easytier. easytier-core gets config from this and web frontend use it as restful api server."
|
||||||
|
|
||||||
|
|||||||
@@ -81,6 +81,7 @@ const bool_flags: BoolFlag[] = [
|
|||||||
{ field: 'latency_first', help: 'latency_first_help' },
|
{ field: 'latency_first', help: 'latency_first_help' },
|
||||||
{ field: 'use_smoltcp', help: 'use_smoltcp_help' },
|
{ field: 'use_smoltcp', help: 'use_smoltcp_help' },
|
||||||
{ field: 'disable_ipv6', help: 'disable_ipv6_help' },
|
{ field: 'disable_ipv6', help: 'disable_ipv6_help' },
|
||||||
|
{ field: 'ipv6_public_addr_auto', help: 'ipv6_public_addr_auto_help' },
|
||||||
{ field: 'enable_kcp_proxy', help: 'enable_kcp_proxy_help' },
|
{ field: 'enable_kcp_proxy', help: 'enable_kcp_proxy_help' },
|
||||||
{ field: 'disable_kcp_input', help: 'disable_kcp_input_help' },
|
{ field: 'disable_kcp_input', help: 'disable_kcp_input_help' },
|
||||||
{ field: 'enable_quic_proxy', help: 'enable_quic_proxy_help' },
|
{ field: 'enable_quic_proxy', help: 'enable_quic_proxy_help' },
|
||||||
@@ -98,6 +99,7 @@ const bool_flags: BoolFlag[] = [
|
|||||||
{ field: 'disable_encryption', help: 'disable_encryption_help' },
|
{ field: 'disable_encryption', help: 'disable_encryption_help' },
|
||||||
{ field: 'disable_tcp_hole_punching', help: 'disable_tcp_hole_punching_help' },
|
{ field: 'disable_tcp_hole_punching', help: 'disable_tcp_hole_punching_help' },
|
||||||
{ field: 'disable_udp_hole_punching', help: 'disable_udp_hole_punching_help' },
|
{ field: 'disable_udp_hole_punching', help: 'disable_udp_hole_punching_help' },
|
||||||
|
{ field: 'disable_upnp', help: 'disable_upnp_help' },
|
||||||
{ field: 'disable_sym_hole_punching', help: 'disable_sym_hole_punching_help' },
|
{ field: 'disable_sym_hole_punching', help: 'disable_sym_hole_punching_help' },
|
||||||
{ field: 'enable_magic_dns', help: 'enable_magic_dns_help' },
|
{ field: 'enable_magic_dns', help: 'enable_magic_dns_help' },
|
||||||
{ field: 'enable_private_mode', help: 'enable_private_mode_help' },
|
{ field: 'enable_private_mode', help: 'enable_private_mode_help' },
|
||||||
|
|||||||
@@ -2,7 +2,7 @@
|
|||||||
import { AutoComplete, Button, Dialog, InputNumber, InputText } from 'primevue'
|
import { AutoComplete, Button, Dialog, InputNumber, InputText } from 'primevue'
|
||||||
import InputGroup from 'primevue/inputgroup'
|
import InputGroup from 'primevue/inputgroup'
|
||||||
import InputGroupAddon from 'primevue/inputgroupaddon'
|
import InputGroupAddon from 'primevue/inputgroupaddon'
|
||||||
import { computed, onMounted, onUnmounted, ref, watch } from 'vue'
|
import { computed, ref, watch } from 'vue'
|
||||||
import { useI18n } from 'vue-i18n'
|
import { useI18n } from 'vue-i18n'
|
||||||
|
|
||||||
const props = defineProps<{
|
const props = defineProps<{
|
||||||
@@ -13,25 +13,8 @@ const props = defineProps<{
|
|||||||
const { t } = useI18n()
|
const { t } = useI18n()
|
||||||
const url = defineModel<string>({ required: true })
|
const url = defineModel<string>({ required: true })
|
||||||
const editing = ref(false)
|
const editing = ref(false)
|
||||||
const container = ref<HTMLElement | null>(null)
|
|
||||||
const internalCompact = ref(false)
|
|
||||||
const hostFocused = ref(false)
|
const hostFocused = ref(false)
|
||||||
|
|
||||||
onMounted(() => {
|
|
||||||
if (container.value) {
|
|
||||||
const observer = new ResizeObserver(entries => {
|
|
||||||
for (const entry of entries) {
|
|
||||||
internalCompact.value = entry.contentRect.width < 400
|
|
||||||
}
|
|
||||||
})
|
|
||||||
observer.observe(container.value)
|
|
||||||
|
|
||||||
onUnmounted(() => {
|
|
||||||
observer.disconnect()
|
|
||||||
})
|
|
||||||
}
|
|
||||||
})
|
|
||||||
|
|
||||||
const parseUrl = (val: string | null | undefined): { proto: string; host: string; port: number | null } => {
|
const parseUrl = (val: string | null | undefined): { proto: string; host: string; port: number | null } => {
|
||||||
const getValidPort = (portStr: string, proto: string) => {
|
const getValidPort = (portStr: string, proto: string) => {
|
||||||
const p = parseInt(portStr)
|
const p = parseInt(portStr)
|
||||||
@@ -169,28 +152,30 @@ const onProtoChange = (newProto: string) => {
|
|||||||
</script>
|
</script>
|
||||||
|
|
||||||
<template>
|
<template>
|
||||||
<div ref="container" class="w-full">
|
<div class="url-input-container w-full min-w-0 overflow-hidden">
|
||||||
<InputGroup v-if="!internalCompact" class="w-full">
|
<InputGroup class="url-input-full w-full min-w-0">
|
||||||
<AutoComplete :model-value="internalValue.proto" :suggestions="filteredProtos" dropdown
|
<AutoComplete :model-value="internalValue.proto" :suggestions="filteredProtos" dropdown
|
||||||
class="max-w-32 proto-autocomplete-in-group" @complete="searchProtos"
|
class="max-w-32 proto-autocomplete-in-group" @complete="searchProtos"
|
||||||
@update:model-value="onProtoChange" />
|
@update:model-value="onProtoChange" />
|
||||||
<InputText v-model="internalValue.host" :placeholder="placeholder || '0.0.0.0'" class="grow"
|
<InputText v-model="internalValue.host" :placeholder="placeholder || '0.0.0.0'" class="grow min-w-0"
|
||||||
@focus="onHostFocus" @blur="onHostBlur" />
|
@focus="onHostFocus" @blur="onHostBlur" />
|
||||||
<template v-if="!isNoPortProto">
|
<template v-if="!isNoPortProto">
|
||||||
<InputGroupAddon>
|
<InputGroupAddon>
|
||||||
<span style="font-weight: bold">:</span>
|
<span style="font-weight: bold">:</span>
|
||||||
</InputGroupAddon>
|
</InputGroupAddon>
|
||||||
<InputNumber v-model="internalValue.port" :format="false" :min="1" :max="65535" class="max-w-24"
|
<InputNumber v-model="internalValue.port" :format="false" :min="1" :max="65535" class="max-w-24"
|
||||||
:placeholder="String(protos[internalValue.proto] ?? 11010)"
|
:placeholder="String(protos[internalValue.proto] ?? 11010)" fluid />
|
||||||
fluid />
|
|
||||||
</template>
|
</template>
|
||||||
|
<!-- Rendered in both responsive branches; keep action slot content free of side effects and duplicate IDs. -->
|
||||||
<slot name="actions"></slot>
|
<slot name="actions"></slot>
|
||||||
</InputGroup>
|
</InputGroup>
|
||||||
|
|
||||||
<div v-else class="flex justify-between items-center p-2 border rounded w-full">
|
<div
|
||||||
<span class="truncate mr-2">{{ url }}</span>
|
class="url-input-compact flex justify-between items-center p-2 border rounded w-full min-w-0 overflow-hidden">
|
||||||
<div class="flex items-center">
|
<span class="truncate mr-2 min-w-0 flex-1 overflow-hidden">{{ url }}</span>
|
||||||
<Button icon="pi pi-pencil" class="p-button-sm p-button-text" @click="editing = true" />
|
<div class="flex items-center shrink-0">
|
||||||
|
<Button icon="pi pi-pencil" class="p-button-sm p-button-text" :aria-label="t('web.common.edit')"
|
||||||
|
@click="editing = true" />
|
||||||
<slot name="actions"></slot>
|
<slot name="actions"></slot>
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
@@ -222,6 +207,28 @@ const onProtoChange = (newProto: string) => {
|
|||||||
</template>
|
</template>
|
||||||
|
|
||||||
<style scoped>
|
<style scoped>
|
||||||
|
.url-input-container {
|
||||||
|
container-type: inline-size;
|
||||||
|
}
|
||||||
|
|
||||||
|
.url-input-full {
|
||||||
|
display: none;
|
||||||
|
}
|
||||||
|
|
||||||
|
.url-input-compact {
|
||||||
|
display: flex;
|
||||||
|
}
|
||||||
|
|
||||||
|
@container (min-width: 400px) {
|
||||||
|
.url-input-full {
|
||||||
|
display: flex;
|
||||||
|
}
|
||||||
|
|
||||||
|
.url-input-compact {
|
||||||
|
display: none;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
.proto-autocomplete-in-group,
|
.proto-autocomplete-in-group,
|
||||||
.proto-autocomplete-in-group :deep(.p-autocomplete-input),
|
.proto-autocomplete-in-group :deep(.p-autocomplete-input),
|
||||||
.proto-autocomplete-in-group :deep(.p-autocomplete-dropdown) {
|
.proto-autocomplete-in-group :deep(.p-autocomplete-dropdown) {
|
||||||
|
|||||||
@@ -104,6 +104,9 @@ use_smoltcp_help: 使用用户态 TCP/IP 协议栈,避免操作系统防火墙
|
|||||||
disable_ipv6: 禁用IPv6
|
disable_ipv6: 禁用IPv6
|
||||||
disable_ipv6_help: 禁用此节点的IPv6功能,仅使用IPv4进行网络通信。
|
disable_ipv6_help: 禁用此节点的IPv6功能,仅使用IPv4进行网络通信。
|
||||||
|
|
||||||
|
ipv6_public_addr_auto: 自动获取公网 IPv6
|
||||||
|
ipv6_public_addr_auto_help: 自动从共享了 IPv6 子网的对等节点获取一个公网 IPv6 地址。
|
||||||
|
|
||||||
enable_kcp_proxy: 启用 KCP 代理
|
enable_kcp_proxy: 启用 KCP 代理
|
||||||
enable_kcp_proxy_help: 将 TCP 流量转为 KCP 流量,降低传输延迟,提升传输速度。
|
enable_kcp_proxy_help: 将 TCP 流量转为 KCP 流量,降低传输延迟,提升传输速度。
|
||||||
|
|
||||||
@@ -157,6 +160,9 @@ disable_tcp_hole_punching_help: 禁用TCP打洞功能
|
|||||||
disable_udp_hole_punching: 禁用UDP打洞
|
disable_udp_hole_punching: 禁用UDP打洞
|
||||||
disable_udp_hole_punching_help: 禁用UDP打洞功能
|
disable_udp_hole_punching_help: 禁用UDP打洞功能
|
||||||
|
|
||||||
|
disable_upnp: 禁用 UPnP
|
||||||
|
disable_upnp_help: 禁用符合条件监听器的运行时 UPnP/NAT-PMP 端口映射;自动端口映射默认开启。
|
||||||
|
|
||||||
disable_sym_hole_punching: 禁用对称NAT打洞
|
disable_sym_hole_punching: 禁用对称NAT打洞
|
||||||
disable_sym_hole_punching_help: 禁用对称NAT的打洞(生日攻击),将对称NAT视为锥形NAT处理
|
disable_sym_hole_punching_help: 禁用对称NAT的打洞(生日攻击),将对称NAT视为锥形NAT处理
|
||||||
|
|
||||||
|
|||||||
@@ -103,6 +103,9 @@ use_smoltcp_help: Use a user-space TCP/IP stack to avoid issues with operating s
|
|||||||
disable_ipv6: Disable IPv6
|
disable_ipv6: Disable IPv6
|
||||||
disable_ipv6_help: Disable IPv6 functionality for this node, only use IPv4 for network communication.
|
disable_ipv6_help: Disable IPv6 functionality for this node, only use IPv4 for network communication.
|
||||||
|
|
||||||
|
ipv6_public_addr_auto: Auto Public IPv6
|
||||||
|
ipv6_public_addr_auto_help: Auto-obtain a public IPv6 address from a peer that shares its IPv6 subnet.
|
||||||
|
|
||||||
enable_kcp_proxy: Enable KCP Proxy
|
enable_kcp_proxy: Enable KCP Proxy
|
||||||
enable_kcp_proxy_help: Convert TCP traffic to KCP traffic to reduce latency and boost transmission speed.
|
enable_kcp_proxy_help: Convert TCP traffic to KCP traffic to reduce latency and boost transmission speed.
|
||||||
|
|
||||||
@@ -156,6 +159,9 @@ disable_tcp_hole_punching_help: Disable tcp hole punching
|
|||||||
disable_udp_hole_punching: Disable UDP Hole Punching
|
disable_udp_hole_punching: Disable UDP Hole Punching
|
||||||
disable_udp_hole_punching_help: Disable udp hole punching
|
disable_udp_hole_punching_help: Disable udp hole punching
|
||||||
|
|
||||||
|
disable_upnp: Disable UPnP
|
||||||
|
disable_upnp_help: Disable runtime UPnP/NAT-PMP port mapping for eligible listeners; automatic port mapping is enabled by default.
|
||||||
|
|
||||||
disable_sym_hole_punching: Disable Symmetric NAT Hole Punching
|
disable_sym_hole_punching: Disable Symmetric NAT Hole Punching
|
||||||
disable_sym_hole_punching_help: Disable special hole punching handling for symmetric NAT (based on birthday attack), treat symmetric NAT as cone NAT
|
disable_sym_hole_punching_help: Disable special hole punching handling for symmetric NAT (based on birthday attack), treat symmetric NAT as cone NAT
|
||||||
|
|
||||||
|
|||||||
@@ -115,6 +115,7 @@ export interface NetworkConfig {
|
|||||||
|
|
||||||
use_smoltcp?: boolean
|
use_smoltcp?: boolean
|
||||||
disable_ipv6?: boolean
|
disable_ipv6?: boolean
|
||||||
|
ipv6_public_addr_auto?: boolean
|
||||||
enable_kcp_proxy?: boolean
|
enable_kcp_proxy?: boolean
|
||||||
disable_kcp_input?: boolean
|
disable_kcp_input?: boolean
|
||||||
enable_quic_proxy?: boolean
|
enable_quic_proxy?: boolean
|
||||||
@@ -132,6 +133,7 @@ export interface NetworkConfig {
|
|||||||
disable_encryption?: boolean
|
disable_encryption?: boolean
|
||||||
disable_tcp_hole_punching?: boolean
|
disable_tcp_hole_punching?: boolean
|
||||||
disable_udp_hole_punching?: boolean
|
disable_udp_hole_punching?: boolean
|
||||||
|
disable_upnp?: boolean
|
||||||
disable_sym_hole_punching?: boolean
|
disable_sym_hole_punching?: boolean
|
||||||
|
|
||||||
enable_relay_network_whitelist?: boolean
|
enable_relay_network_whitelist?: boolean
|
||||||
@@ -190,6 +192,7 @@ export function DEFAULT_NETWORK_CONFIG(): NetworkConfig {
|
|||||||
|
|
||||||
use_smoltcp: false,
|
use_smoltcp: false,
|
||||||
disable_ipv6: false,
|
disable_ipv6: false,
|
||||||
|
ipv6_public_addr_auto: false,
|
||||||
enable_kcp_proxy: false,
|
enable_kcp_proxy: false,
|
||||||
disable_kcp_input: false,
|
disable_kcp_input: false,
|
||||||
enable_quic_proxy: false,
|
enable_quic_proxy: false,
|
||||||
@@ -207,6 +210,7 @@ export function DEFAULT_NETWORK_CONFIG(): NetworkConfig {
|
|||||||
disable_encryption: false,
|
disable_encryption: false,
|
||||||
disable_tcp_hole_punching: false,
|
disable_tcp_hole_punching: false,
|
||||||
disable_udp_hole_punching: false,
|
disable_udp_hole_punching: false,
|
||||||
|
disable_upnp: false,
|
||||||
disable_sym_hole_punching: false,
|
disable_sym_hole_punching: false,
|
||||||
enable_relay_network_whitelist: false,
|
enable_relay_network_whitelist: false,
|
||||||
relay_network_whitelist: [],
|
relay_network_whitelist: [],
|
||||||
|
|||||||
+1
-1
@@ -3,7 +3,7 @@ name = "easytier"
|
|||||||
description = "A full meshed p2p VPN, connecting all your devices in one network with one command."
|
description = "A full meshed p2p VPN, connecting all your devices in one network with one command."
|
||||||
homepage = "https://github.com/EasyTier/EasyTier"
|
homepage = "https://github.com/EasyTier/EasyTier"
|
||||||
repository = "https://github.com/EasyTier/EasyTier"
|
repository = "https://github.com/EasyTier/EasyTier"
|
||||||
version = "2.6.2"
|
version = "2.6.3"
|
||||||
edition.workspace = true
|
edition.workspace = true
|
||||||
rust-version.workspace = true
|
rust-version.workspace = true
|
||||||
authors = ["kkrainbow"]
|
authors = ["kkrainbow"]
|
||||||
|
|||||||
@@ -137,12 +137,13 @@ pub fn setup_socket_for_win<S: AsRawSocket>(
|
|||||||
}
|
}
|
||||||
|
|
||||||
let socket = SOCKET(socket.as_raw_socket() as usize);
|
let socket = SOCKET(socket.as_raw_socket() as usize);
|
||||||
let optval = 1_i32.to_ne_bytes();
|
|
||||||
unsafe {
|
// let optval = 1_i32.to_ne_bytes();
|
||||||
if setsockopt(socket, SOL_SOCKET, SO_EXCLUSIVEADDRUSE, Some(&optval)) == SOCKET_ERROR {
|
// unsafe {
|
||||||
return Err(io::Error::last_os_error());
|
// if setsockopt(socket, SOL_SOCKET, SO_EXCLUSIVEADDRUSE, Some(&optval)) == SOCKET_ERROR {
|
||||||
}
|
// return Err(io::Error::last_os_error());
|
||||||
}
|
// }
|
||||||
|
// }
|
||||||
|
|
||||||
if let Some(iface) = bind_dev {
|
if let Some(iface) = bind_dev {
|
||||||
set_ip_unicast_if(socket, bind_addr, &iface)?;
|
set_ip_unicast_if(socket, bind_addr, &iface)?;
|
||||||
|
|||||||
@@ -39,6 +39,7 @@ pub struct RateLimitValue {
|
|||||||
pub enum RuleId {
|
pub enum RuleId {
|
||||||
Priority(u32),
|
Priority(u32),
|
||||||
Stateful(u32),
|
Stateful(u32),
|
||||||
|
StatefulReverse,
|
||||||
Default,
|
Default,
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -48,6 +49,7 @@ impl RuleId {
|
|||||||
match self {
|
match self {
|
||||||
RuleId::Priority(p) => p.to_string(),
|
RuleId::Priority(p) => p.to_string(),
|
||||||
RuleId::Stateful(p) => format!("stateful-{}", p),
|
RuleId::Stateful(p) => format!("stateful-{}", p),
|
||||||
|
RuleId::StatefulReverse => "stateful-reverse".to_string(),
|
||||||
RuleId::Default => "default".to_string(),
|
RuleId::Default => "default".to_string(),
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -482,20 +484,30 @@ impl AclProcessor {
|
|||||||
stats
|
stats
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Process a packet through ACL rules - Now lock-free!
|
/// Process a packet through ACL rules.
|
||||||
pub fn process_packet(&self, packet_info: &PacketInfo, chain_type: ChainType) -> AclResult {
|
pub fn process_packet(&self, packet_info: &PacketInfo, chain_type: ChainType) -> AclResult {
|
||||||
// Check cache first for performance
|
// Check cache first for performance
|
||||||
let cache_key = AclCacheKey::from_packet_info(packet_info, chain_type);
|
let cache_key = AclCacheKey::from_packet_info(packet_info, chain_type);
|
||||||
|
|
||||||
// If cache hit and can skip checks, return cached result
|
// If cache hit and can skip checks, return cached result. Cached drops may be
|
||||||
|
// overridden by a stateful reverse connection that was created after caching.
|
||||||
if let Some(mut cached) = self.rule_cache.get_mut(&cache_key) {
|
if let Some(mut cached) = self.rule_cache.get_mut(&cache_key) {
|
||||||
// Update last access time for LRU
|
// Update last access time for LRU
|
||||||
cached.last_access = Instant::now();
|
cached.last_access = Instant::now();
|
||||||
|
|
||||||
self.increment_stat(AclStatKey::CacheHits);
|
self.increment_stat(AclStatKey::CacheHits);
|
||||||
|
if cached.acl_result.as_ref().map(|r| r.action) == Some(Action::Drop)
|
||||||
|
&& let Some(result) = self.check_reverse_connection(packet_info)
|
||||||
|
{
|
||||||
|
return result;
|
||||||
|
}
|
||||||
return self.process_packet_with_cache_entry(packet_info, &cached);
|
return self.process_packet_with_cache_entry(packet_info, &cached);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if let Some(result) = self.check_reverse_connection(packet_info) {
|
||||||
|
return result;
|
||||||
|
}
|
||||||
|
|
||||||
// Direct access to rules - no locks needed!
|
// Direct access to rules - no locks needed!
|
||||||
let rules = match chain_type {
|
let rules = match chain_type {
|
||||||
ChainType::Inbound => &self.inbound_rules,
|
ChainType::Inbound => &self.inbound_rules,
|
||||||
@@ -730,28 +742,68 @@ impl AclProcessor {
|
|||||||
}
|
}
|
||||||
|
|
||||||
fn conn_track_key(&self, packet_info: &PacketInfo) -> String {
|
fn conn_track_key(&self, packet_info: &PacketInfo) -> String {
|
||||||
|
Self::make_conn_track_key(
|
||||||
|
packet_info.src_ip,
|
||||||
|
packet_info.src_port,
|
||||||
|
packet_info.dst_ip,
|
||||||
|
packet_info.dst_port,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
|
fn reverse_conn_track_key(&self, packet_info: &PacketInfo) -> String {
|
||||||
|
Self::make_conn_track_key(
|
||||||
|
packet_info.dst_ip,
|
||||||
|
packet_info.dst_port,
|
||||||
|
packet_info.src_ip,
|
||||||
|
packet_info.src_port,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
|
fn make_conn_track_key(
|
||||||
|
src_ip: IpAddr,
|
||||||
|
src_port: Option<u16>,
|
||||||
|
dst_ip: IpAddr,
|
||||||
|
dst_port: Option<u16>,
|
||||||
|
) -> String {
|
||||||
format!(
|
format!(
|
||||||
"{}:{}->{}:{}",
|
"{}:{}->{}:{}",
|
||||||
packet_info.src_ip,
|
src_ip,
|
||||||
packet_info.src_port.unwrap_or(0),
|
src_port.unwrap_or(0),
|
||||||
packet_info.dst_ip,
|
dst_ip,
|
||||||
packet_info.dst_port.unwrap_or(0)
|
dst_port.unwrap_or(0)
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn check_reverse_connection(&self, packet_info: &PacketInfo) -> Option<AclResult> {
|
||||||
|
let reverse_key = self.reverse_conn_track_key(packet_info);
|
||||||
|
let mut entry = self.conn_track.get_mut(&reverse_key)?;
|
||||||
|
Self::update_conn_track_entry(entry.value_mut(), packet_info);
|
||||||
|
Some(AclResult {
|
||||||
|
action: Action::Allow,
|
||||||
|
matched_rule: Some(RuleId::StatefulReverse),
|
||||||
|
should_log: false,
|
||||||
|
log_context: Some(AclLogContext::StatefulMatch {
|
||||||
|
src_ip: packet_info.src_ip,
|
||||||
|
dst_ip: packet_info.dst_ip,
|
||||||
|
}),
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
fn update_conn_track_entry(entry: &mut ConnTrackEntry, packet_info: &PacketInfo) {
|
||||||
|
entry.last_seen = SystemTime::now()
|
||||||
|
.duration_since(UNIX_EPOCH)
|
||||||
|
.unwrap()
|
||||||
|
.as_secs();
|
||||||
|
entry.packet_count += 1;
|
||||||
|
entry.byte_count += packet_info.packet_size as u64;
|
||||||
|
entry.state = ConnState::Established as i32;
|
||||||
|
}
|
||||||
|
|
||||||
/// Check connection state for stateful rules
|
/// Check connection state for stateful rules
|
||||||
fn check_connection_state(&self, conn_track_key: &str, packet_info: &PacketInfo) {
|
fn check_connection_state(&self, conn_track_key: &str, packet_info: &PacketInfo) {
|
||||||
self.conn_track
|
self.conn_track
|
||||||
.entry(conn_track_key.to_string())
|
.entry(conn_track_key.to_string())
|
||||||
.and_modify(|x| {
|
.and_modify(|x| Self::update_conn_track_entry(x, packet_info))
|
||||||
x.last_seen = SystemTime::now()
|
|
||||||
.duration_since(UNIX_EPOCH)
|
|
||||||
.unwrap()
|
|
||||||
.as_secs();
|
|
||||||
x.packet_count += 1;
|
|
||||||
x.byte_count += packet_info.packet_size as u64;
|
|
||||||
x.state = ConnState::Established as i32;
|
|
||||||
})
|
|
||||||
.or_insert_with(|| ConnTrackEntry {
|
.or_insert_with(|| ConnTrackEntry {
|
||||||
src_addr: Some(
|
src_addr: Some(
|
||||||
SocketAddr::new(packet_info.src_ip, packet_info.src_port.unwrap_or(0)).into(),
|
SocketAddr::new(packet_info.src_ip, packet_info.src_port.unwrap_or(0)).into(),
|
||||||
@@ -1382,6 +1434,73 @@ mod tests {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn test_stateful_allows_reverse_traffic_before_default_drop() {
|
||||||
|
let runtime = tokio::runtime::Runtime::new().unwrap();
|
||||||
|
let _runtime_guard = runtime.enter();
|
||||||
|
|
||||||
|
let mut acl_config = Acl::default();
|
||||||
|
let mut acl_v1 = AclV1::default();
|
||||||
|
|
||||||
|
let mut outbound_chain = Chain {
|
||||||
|
name: "outbound_stateful".to_string(),
|
||||||
|
chain_type: ChainType::Outbound as i32,
|
||||||
|
enabled: true,
|
||||||
|
default_action: Action::Drop as i32,
|
||||||
|
..Default::default()
|
||||||
|
};
|
||||||
|
outbound_chain.rules.push(Rule {
|
||||||
|
name: "allow_out_stateful".to_string(),
|
||||||
|
priority: 100,
|
||||||
|
enabled: true,
|
||||||
|
action: Action::Allow as i32,
|
||||||
|
protocol: Protocol::Tcp as i32,
|
||||||
|
stateful: true,
|
||||||
|
..Default::default()
|
||||||
|
});
|
||||||
|
|
||||||
|
let inbound_chain = Chain {
|
||||||
|
name: "inbound_default_drop".to_string(),
|
||||||
|
chain_type: ChainType::Inbound as i32,
|
||||||
|
enabled: true,
|
||||||
|
default_action: Action::Drop as i32,
|
||||||
|
..Default::default()
|
||||||
|
};
|
||||||
|
|
||||||
|
acl_v1.chains.push(outbound_chain);
|
||||||
|
acl_v1.chains.push(inbound_chain);
|
||||||
|
acl_config.acl_v1 = Some(acl_v1);
|
||||||
|
|
||||||
|
let processor = AclProcessor::new(acl_config);
|
||||||
|
let outbound_packet = PacketInfo {
|
||||||
|
src_ip: IpAddr::V4(Ipv4Addr::new(10, 0, 0, 1)),
|
||||||
|
dst_ip: IpAddr::V4(Ipv4Addr::new(10, 0, 0, 2)),
|
||||||
|
src_port: Some(12345),
|
||||||
|
dst_port: Some(22),
|
||||||
|
protocol: Protocol::Tcp,
|
||||||
|
packet_size: 64,
|
||||||
|
src_groups: Arc::new(vec![]),
|
||||||
|
dst_groups: Arc::new(vec![]),
|
||||||
|
};
|
||||||
|
let inbound_reply = PacketInfo {
|
||||||
|
src_ip: outbound_packet.dst_ip,
|
||||||
|
dst_ip: outbound_packet.src_ip,
|
||||||
|
src_port: outbound_packet.dst_port,
|
||||||
|
dst_port: outbound_packet.src_port,
|
||||||
|
protocol: outbound_packet.protocol,
|
||||||
|
packet_size: 64,
|
||||||
|
src_groups: Arc::new(vec![]),
|
||||||
|
dst_groups: Arc::new(vec![]),
|
||||||
|
};
|
||||||
|
|
||||||
|
let outbound_result = processor.process_packet(&outbound_packet, ChainType::Outbound);
|
||||||
|
assert_eq!(outbound_result.action, Action::Allow);
|
||||||
|
|
||||||
|
let inbound_result = processor.process_packet(&inbound_reply, ChainType::Inbound);
|
||||||
|
assert_eq!(inbound_result.action, Action::Allow);
|
||||||
|
assert_eq!(inbound_result.matched_rule, Some(RuleId::StatefulReverse));
|
||||||
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
fn test_acl_cache_key_creation() {
|
fn test_acl_cache_key_creation() {
|
||||||
let packet_info = create_test_packet_info();
|
let packet_info = create_test_packet_info();
|
||||||
|
|||||||
@@ -71,6 +71,7 @@ pub fn gen_default_flags() -> Flags {
|
|||||||
need_p2p: false,
|
need_p2p: false,
|
||||||
instance_recv_bps_limit: u64::MAX,
|
instance_recv_bps_limit: u64::MAX,
|
||||||
disable_upnp: false,
|
disable_upnp: false,
|
||||||
|
disable_relay_data: false,
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
+19
-10
@@ -73,16 +73,6 @@ pub async fn socket_addrs(
|
|||||||
.port()
|
.port()
|
||||||
.or_else(default_port_number)
|
.or_else(default_port_number)
|
||||||
.ok_or(Error::InvalidUrl(url.to_string()))?;
|
.ok_or(Error::InvalidUrl(url.to_string()))?;
|
||||||
// See https://github.com/EasyTier/EasyTier/pull/947
|
|
||||||
// here is for compatibility with old version
|
|
||||||
let port = match port {
|
|
||||||
0 => match url.scheme() {
|
|
||||||
"ws" => 80,
|
|
||||||
"wss" => 443,
|
|
||||||
_ => port,
|
|
||||||
},
|
|
||||||
_ => port,
|
|
||||||
};
|
|
||||||
|
|
||||||
// if host is an ip address, return it directly
|
// if host is an ip address, return it directly
|
||||||
match host {
|
match host {
|
||||||
@@ -139,4 +129,23 @@ mod tests {
|
|||||||
assert_eq!(2, addrs.len(), "addrs: {:?}", addrs);
|
assert_eq!(2, addrs.len(), "addrs: {:?}", addrs);
|
||||||
println!("addrs2: {:?}", addrs);
|
println!("addrs2: {:?}", addrs);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn socket_addrs_preserves_explicit_zero_port() {
|
||||||
|
let cases = [
|
||||||
|
("ws://127.0.0.1:0", 80, 0),
|
||||||
|
("wss://127.0.0.1:0", 443, 0),
|
||||||
|
("ws://127.0.0.1", 80, 80),
|
||||||
|
("wss://127.0.0.1", 443, 443),
|
||||||
|
];
|
||||||
|
|
||||||
|
for (raw_url, default_port, expected_port) in cases {
|
||||||
|
let url = url::Url::parse(raw_url).unwrap();
|
||||||
|
let addrs = socket_addrs(&url, || Some(default_port)).await.unwrap();
|
||||||
|
assert_eq!(
|
||||||
|
addrs,
|
||||||
|
vec![SocketAddr::from(([127, 0, 0, 1], expected_port))]
|
||||||
|
);
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -217,6 +217,12 @@ pub struct GlobalCtx {
|
|||||||
|
|
||||||
flags: ArcSwap<Flags>,
|
flags: ArcSwap<Flags>,
|
||||||
|
|
||||||
|
// Runtime/base advertised feature flags before config-owned fields are
|
||||||
|
// overlaid by set_flags. Keep this separate so config patches do not erase
|
||||||
|
// runtime state such as public-server role, IPv6 provider status, or the
|
||||||
|
// non-whitelist avoid-relay preference.
|
||||||
|
base_feature_flags: AtomicCell<PeerFeatureFlag>,
|
||||||
|
|
||||||
feature_flags: AtomicCell<PeerFeatureFlag>,
|
feature_flags: AtomicCell<PeerFeatureFlag>,
|
||||||
|
|
||||||
token_bucket_manager: TokenBucketManager,
|
token_bucket_manager: TokenBucketManager,
|
||||||
@@ -247,8 +253,17 @@ impl std::fmt::Debug for GlobalCtx {
|
|||||||
pub type ArcGlobalCtx = std::sync::Arc<GlobalCtx>;
|
pub type ArcGlobalCtx = std::sync::Arc<GlobalCtx>;
|
||||||
|
|
||||||
impl GlobalCtx {
|
impl GlobalCtx {
|
||||||
fn derive_feature_flags(flags: &Flags, current: Option<PeerFeatureFlag>) -> PeerFeatureFlag {
|
fn apply_disable_relay_data_flag(
|
||||||
let mut feature_flags = current.unwrap_or_default();
|
flags: &Flags,
|
||||||
|
mut feature_flags: PeerFeatureFlag,
|
||||||
|
) -> PeerFeatureFlag {
|
||||||
|
if flags.disable_relay_data {
|
||||||
|
feature_flags.avoid_relay_data = true;
|
||||||
|
}
|
||||||
|
feature_flags
|
||||||
|
}
|
||||||
|
|
||||||
|
fn derive_feature_flags(flags: &Flags, mut feature_flags: PeerFeatureFlag) -> PeerFeatureFlag {
|
||||||
feature_flags.kcp_input = !flags.disable_kcp_input;
|
feature_flags.kcp_input = !flags.disable_kcp_input;
|
||||||
feature_flags.no_relay_kcp = flags.disable_relay_kcp;
|
feature_flags.no_relay_kcp = flags.disable_relay_kcp;
|
||||||
feature_flags.support_conn_list_sync = true;
|
feature_flags.support_conn_list_sync = true;
|
||||||
@@ -256,7 +271,7 @@ impl GlobalCtx {
|
|||||||
feature_flags.no_relay_quic = flags.disable_relay_quic;
|
feature_flags.no_relay_quic = flags.disable_relay_quic;
|
||||||
feature_flags.need_p2p = flags.need_p2p;
|
feature_flags.need_p2p = flags.need_p2p;
|
||||||
feature_flags.disable_p2p = flags.disable_p2p;
|
feature_flags.disable_p2p = flags.disable_p2p;
|
||||||
feature_flags
|
Self::apply_disable_relay_data_flag(flags, feature_flags)
|
||||||
}
|
}
|
||||||
|
|
||||||
pub fn new(config_fs: impl ConfigLoader + 'static) -> Self {
|
pub fn new(config_fs: impl ConfigLoader + 'static) -> Self {
|
||||||
@@ -285,7 +300,8 @@ impl GlobalCtx {
|
|||||||
|
|
||||||
let flags = config_fs.get_flags();
|
let flags = config_fs.get_flags();
|
||||||
|
|
||||||
let feature_flags = Self::derive_feature_flags(&flags, None);
|
let base_feature_flags = PeerFeatureFlag::default();
|
||||||
|
let feature_flags = Self::derive_feature_flags(&flags, base_feature_flags);
|
||||||
|
|
||||||
let credential_storage_path = config_fs.get_credential_file();
|
let credential_storage_path = config_fs.get_credential_file();
|
||||||
let credential_manager = Arc::new(CredentialManager::new(credential_storage_path));
|
let credential_manager = Arc::new(CredentialManager::new(credential_storage_path));
|
||||||
@@ -318,6 +334,8 @@ impl GlobalCtx {
|
|||||||
|
|
||||||
flags: ArcSwap::new(Arc::new(flags)),
|
flags: ArcSwap::new(Arc::new(flags)),
|
||||||
|
|
||||||
|
base_feature_flags: AtomicCell::new(base_feature_flags),
|
||||||
|
|
||||||
feature_flags: AtomicCell::new(feature_flags),
|
feature_flags: AtomicCell::new(feature_flags),
|
||||||
|
|
||||||
token_bucket_manager: TokenBucketManager::new(),
|
token_bucket_manager: TokenBucketManager::new(),
|
||||||
@@ -513,7 +531,7 @@ impl GlobalCtx {
|
|||||||
self.config.set_flags(flags.clone());
|
self.config.set_flags(flags.clone());
|
||||||
self.feature_flags.store(Self::derive_feature_flags(
|
self.feature_flags.store(Self::derive_feature_flags(
|
||||||
&flags,
|
&flags,
|
||||||
Some(self.feature_flags.load()),
|
self.base_feature_flags.load(),
|
||||||
));
|
));
|
||||||
self.flags.store(Arc::new(flags));
|
self.flags.store(Arc::new(flags));
|
||||||
}
|
}
|
||||||
@@ -578,8 +596,53 @@ impl GlobalCtx {
|
|||||||
self.feature_flags.load()
|
self.feature_flags.load()
|
||||||
}
|
}
|
||||||
|
|
||||||
pub fn set_feature_flags(&self, flags: PeerFeatureFlag) {
|
/// Replace the runtime/base advertised flags as a complete snapshot.
|
||||||
self.feature_flags.store(flags);
|
///
|
||||||
|
/// This is intended for foreign scoped contexts that inherit an already
|
||||||
|
/// computed feature-flag snapshot from their parent. Most callers should use
|
||||||
|
/// a narrower setter so they do not accidentally overwrite unrelated runtime
|
||||||
|
/// state.
|
||||||
|
pub fn set_base_advertised_feature_flags(&self, feature_flags: PeerFeatureFlag) {
|
||||||
|
self.base_feature_flags.store(feature_flags);
|
||||||
|
let flags = self.flags.load();
|
||||||
|
self.feature_flags
|
||||||
|
.store(Self::apply_disable_relay_data_flag(
|
||||||
|
flags.as_ref(),
|
||||||
|
feature_flags,
|
||||||
|
));
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Set the avoid-relay preference that is independent of disable_relay_data.
|
||||||
|
///
|
||||||
|
/// disable_relay_data still forces the effective advertised flag to true,
|
||||||
|
/// but this base preference is preserved when that config flag is toggled.
|
||||||
|
pub fn set_avoid_relay_data_preference(&self, avoid_relay_data: bool) -> bool {
|
||||||
|
let mut base_feature_flags = self.base_feature_flags.load();
|
||||||
|
base_feature_flags.avoid_relay_data = avoid_relay_data;
|
||||||
|
self.base_feature_flags.store(base_feature_flags);
|
||||||
|
|
||||||
|
let mut feature_flags = self.feature_flags.load();
|
||||||
|
let previous = feature_flags.avoid_relay_data;
|
||||||
|
feature_flags.avoid_relay_data = avoid_relay_data || self.flags.load().disable_relay_data;
|
||||||
|
self.feature_flags.store(feature_flags);
|
||||||
|
previous != feature_flags.avoid_relay_data
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Set the runtime IPv6-provider advertised bit without touching
|
||||||
|
/// config-derived feature flags.
|
||||||
|
pub fn set_ipv6_public_addr_provider_feature_flag(&self, enabled: bool) -> bool {
|
||||||
|
let mut base_feature_flags = self.base_feature_flags.load();
|
||||||
|
base_feature_flags.ipv6_public_addr_provider = enabled;
|
||||||
|
self.base_feature_flags.store(base_feature_flags);
|
||||||
|
|
||||||
|
let mut feature_flags = self.feature_flags.load();
|
||||||
|
if feature_flags.ipv6_public_addr_provider == enabled {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
feature_flags.ipv6_public_addr_provider = enabled;
|
||||||
|
self.feature_flags.store(feature_flags);
|
||||||
|
true
|
||||||
}
|
}
|
||||||
|
|
||||||
pub fn token_bucket_manager(&self) -> &TokenBucketManager {
|
pub fn token_bucket_manager(&self) -> &TokenBucketManager {
|
||||||
@@ -796,7 +859,7 @@ pub mod tests {
|
|||||||
let mut feature_flags = global_ctx.get_feature_flags();
|
let mut feature_flags = global_ctx.get_feature_flags();
|
||||||
feature_flags.avoid_relay_data = true;
|
feature_flags.avoid_relay_data = true;
|
||||||
feature_flags.is_public_server = true;
|
feature_flags.is_public_server = true;
|
||||||
global_ctx.set_feature_flags(feature_flags);
|
global_ctx.set_base_advertised_feature_flags(feature_flags);
|
||||||
|
|
||||||
let mut flags = global_ctx.get_flags().clone();
|
let mut flags = global_ctx.get_flags().clone();
|
||||||
flags.disable_kcp_input = true;
|
flags.disable_kcp_input = true;
|
||||||
@@ -820,6 +883,83 @@ pub mod tests {
|
|||||||
assert!(!feature_flags.ipv6_public_addr_provider);
|
assert!(!feature_flags.ipv6_public_addr_provider);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn set_base_advertised_feature_flags_applies_current_values() {
|
||||||
|
let config = TomlConfigLoader::default();
|
||||||
|
let global_ctx = GlobalCtx::new(config);
|
||||||
|
|
||||||
|
let feature_flags = PeerFeatureFlag {
|
||||||
|
kcp_input: false,
|
||||||
|
no_relay_kcp: true,
|
||||||
|
quic_input: false,
|
||||||
|
no_relay_quic: true,
|
||||||
|
is_public_server: true,
|
||||||
|
..Default::default()
|
||||||
|
};
|
||||||
|
global_ctx.set_base_advertised_feature_flags(feature_flags);
|
||||||
|
|
||||||
|
assert_eq!(global_ctx.get_feature_flags(), feature_flags);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn set_base_advertised_feature_flags_keeps_disable_relay_data_effective() {
|
||||||
|
let config = TomlConfigLoader::default();
|
||||||
|
let global_ctx = GlobalCtx::new(config);
|
||||||
|
|
||||||
|
let mut flags = global_ctx.get_flags().clone();
|
||||||
|
flags.disable_relay_data = true;
|
||||||
|
global_ctx.set_flags(flags);
|
||||||
|
|
||||||
|
let mut feature_flags = global_ctx.get_feature_flags();
|
||||||
|
feature_flags.avoid_relay_data = false;
|
||||||
|
feature_flags.is_public_server = true;
|
||||||
|
global_ctx.set_base_advertised_feature_flags(feature_flags);
|
||||||
|
|
||||||
|
let advertised_feature_flags = global_ctx.get_feature_flags();
|
||||||
|
assert!(advertised_feature_flags.avoid_relay_data);
|
||||||
|
assert!(advertised_feature_flags.is_public_server);
|
||||||
|
|
||||||
|
let mut flags = global_ctx.get_flags().clone();
|
||||||
|
flags.disable_relay_data = false;
|
||||||
|
global_ctx.set_flags(flags);
|
||||||
|
|
||||||
|
let advertised_feature_flags = global_ctx.get_feature_flags();
|
||||||
|
assert!(!advertised_feature_flags.avoid_relay_data);
|
||||||
|
assert!(advertised_feature_flags.is_public_server);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn disable_relay_data_sets_avoid_relay_feature_flag() {
|
||||||
|
let config = TomlConfigLoader::default();
|
||||||
|
let global_ctx = GlobalCtx::new(config);
|
||||||
|
|
||||||
|
let mut flags = global_ctx.get_flags().clone();
|
||||||
|
flags.disable_relay_data = true;
|
||||||
|
global_ctx.set_flags(flags);
|
||||||
|
|
||||||
|
assert!(global_ctx.get_feature_flags().avoid_relay_data);
|
||||||
|
|
||||||
|
let mut flags = global_ctx.get_flags().clone();
|
||||||
|
flags.disable_relay_data = false;
|
||||||
|
global_ctx.set_flags(flags);
|
||||||
|
|
||||||
|
assert!(!global_ctx.get_feature_flags().avoid_relay_data);
|
||||||
|
|
||||||
|
global_ctx.set_avoid_relay_data_preference(true);
|
||||||
|
|
||||||
|
let mut flags = global_ctx.get_flags().clone();
|
||||||
|
flags.disable_relay_data = true;
|
||||||
|
global_ctx.set_flags(flags);
|
||||||
|
|
||||||
|
assert!(global_ctx.get_feature_flags().avoid_relay_data);
|
||||||
|
|
||||||
|
let mut flags = global_ctx.get_flags().clone();
|
||||||
|
flags.disable_relay_data = false;
|
||||||
|
global_ctx.set_flags(flags);
|
||||||
|
|
||||||
|
assert!(global_ctx.get_feature_flags().avoid_relay_data);
|
||||||
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
async fn should_deny_proxy_for_process_wide_rpc_port() {
|
async fn should_deny_proxy_for_process_wide_rpc_port() {
|
||||||
protected_port::clear_protected_tcp_ports_for_test();
|
protected_port::clear_protected_tcp_ports_for_test();
|
||||||
|
|||||||
+192
-10
@@ -58,6 +58,21 @@ fn parse_env_filter(default_level: Option<LevelFilter>) -> Result<EnvFilter, any
|
|||||||
.with_context(|| "failed to create env filter")
|
.with_context(|| "failed to create env filter")
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn parse_static_filter(level: LevelFilter) -> Result<EnvFilter, anyhow::Error> {
|
||||||
|
EnvFilter::builder()
|
||||||
|
.with_default_directive(level.into())
|
||||||
|
.parse("")
|
||||||
|
.with_context(|| "failed to create static filter")
|
||||||
|
}
|
||||||
|
|
||||||
|
fn parse_file_filter(level: LevelFilter) -> Result<EnvFilter, anyhow::Error> {
|
||||||
|
if matches!(level, LevelFilter::OFF) {
|
||||||
|
parse_static_filter(level)
|
||||||
|
} else {
|
||||||
|
parse_env_filter(Some(level))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
fn is_log(meta: &Metadata) -> bool {
|
fn is_log(meta: &Metadata) -> bool {
|
||||||
meta.target() == LOG_TARGET || meta.target().starts_with(&format!("{LOG_TARGET}::"))
|
meta.target() == LOG_TARGET || meta.target().starts_with(&format!("{LOG_TARGET}::"))
|
||||||
}
|
}
|
||||||
@@ -165,14 +180,17 @@ fn file_layers(
|
|||||||
) -> anyhow::Result<(Vec<BoxLayer>, Option<NewFilterSender>)> {
|
) -> anyhow::Result<(Vec<BoxLayer>, Option<NewFilterSender>)> {
|
||||||
let mut layers = Vec::new();
|
let mut layers = Vec::new();
|
||||||
|
|
||||||
let level = config.level.map(|s| s.parse().unwrap());
|
let level = config
|
||||||
|
.level
|
||||||
|
.map(|s| s.parse().unwrap())
|
||||||
|
.unwrap_or(LevelFilter::OFF);
|
||||||
|
|
||||||
if matches!(level, Some(LevelFilter::OFF)) && !reload {
|
if matches!(level, LevelFilter::OFF) && !reload {
|
||||||
return Ok((layers, None));
|
return Ok((layers, None));
|
||||||
}
|
}
|
||||||
|
|
||||||
let (file_filter, file_filter_reloader) =
|
let (file_filter, file_filter_reloader) =
|
||||||
tracing_subscriber::reload::Layer::<_, Registry>::new(parse_env_filter(level)?);
|
tracing_subscriber::reload::Layer::<_, Registry>::new(parse_file_filter(level)?);
|
||||||
|
|
||||||
let layer = |wrapper| {
|
let layer = |wrapper| {
|
||||||
layer()
|
layer()
|
||||||
@@ -218,9 +236,7 @@ fn file_layers(
|
|||||||
|
|
||||||
// 初始化全局状态
|
// 初始化全局状态
|
||||||
let _ = LOGGER_LEVEL_SENDER.set(std::sync::Mutex::new(tx.clone()));
|
let _ = LOGGER_LEVEL_SENDER.set(std::sync::Mutex::new(tx.clone()));
|
||||||
if let Some(level) = level {
|
|
||||||
let _ = CURRENT_LOG_LEVEL.set(std::sync::Mutex::new(level.to_string()));
|
let _ = CURRENT_LOG_LEVEL.set(std::sync::Mutex::new(level.to_string()));
|
||||||
}
|
|
||||||
|
|
||||||
std::thread::spawn(move || {
|
std::thread::spawn(move || {
|
||||||
while let Ok(lf) = rx.recv() {
|
while let Ok(lf) = rx.recv() {
|
||||||
@@ -232,11 +248,7 @@ fn file_layers(
|
|||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
let mut new_filter = match EnvFilter::builder()
|
let mut new_filter = match parse_file_filter(parsed_level) {
|
||||||
.with_default_directive(parsed_level.into())
|
|
||||||
.from_env()
|
|
||||||
.with_context(|| "failed to create file filter")
|
|
||||||
{
|
|
||||||
Ok(filter) => Some(filter),
|
Ok(filter) => Some(filter),
|
||||||
Err(e) => {
|
Err(e) => {
|
||||||
error!("Failed to build new log filter for {:?}: {:?}", lf, e);
|
error!("Failed to build new log filter for {:?}: {:?}", lf, e);
|
||||||
@@ -268,6 +280,36 @@ mod tests {
|
|||||||
use super::*;
|
use super::*;
|
||||||
use crate::common::config::FileLoggerConfig;
|
use crate::common::config::FileLoggerConfig;
|
||||||
|
|
||||||
|
const RUST_LOG: &str = "RUST_LOG";
|
||||||
|
|
||||||
|
struct EnvVarGuard {
|
||||||
|
key: &'static str,
|
||||||
|
previous: Option<std::ffi::OsString>,
|
||||||
|
}
|
||||||
|
|
||||||
|
impl EnvVarGuard {
|
||||||
|
fn set(key: &'static str, value: &str) -> Self {
|
||||||
|
let previous = std::env::var_os(key);
|
||||||
|
unsafe { std::env::set_var(key, value) };
|
||||||
|
Self { key, previous }
|
||||||
|
}
|
||||||
|
|
||||||
|
fn unset(key: &'static str) -> Self {
|
||||||
|
let previous = std::env::var_os(key);
|
||||||
|
unsafe { std::env::remove_var(key) };
|
||||||
|
Self { key, previous }
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
impl Drop for EnvVarGuard {
|
||||||
|
fn drop(&mut self) {
|
||||||
|
match &self.previous {
|
||||||
|
Some(value) => unsafe { std::env::set_var(self.key, value) },
|
||||||
|
None => unsafe { std::env::remove_var(self.key) },
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
#[ctor::ctor]
|
#[ctor::ctor]
|
||||||
fn init() {
|
fn init() {
|
||||||
let _ = Registry::default()
|
let _ = Registry::default()
|
||||||
@@ -276,7 +318,147 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
|
fn default_file_logger_level_is_off_without_reload() {
|
||||||
|
let (layers, sender) = file_layers(FileLoggerConfig::default(), false).unwrap();
|
||||||
|
|
||||||
|
assert!(layers.is_empty());
|
||||||
|
assert!(sender.is_none());
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
#[serial_test::serial]
|
||||||
|
fn default_file_logger_level_filters_info_with_reload() {
|
||||||
|
let _guard = EnvVarGuard::set(RUST_LOG, "info");
|
||||||
|
let temp_dir = tempfile::tempdir().unwrap();
|
||||||
|
let log_file_name = "default-off-test.log".to_string();
|
||||||
|
let log_path = temp_dir.path().join(&log_file_name);
|
||||||
|
|
||||||
|
let cfg = FileLoggerConfig {
|
||||||
|
file: Some(log_file_name),
|
||||||
|
dir: Some(temp_dir.path().to_string_lossy().to_string()),
|
||||||
|
..Default::default()
|
||||||
|
};
|
||||||
|
|
||||||
|
let (layers, _sender) = file_layers(cfg, true).unwrap();
|
||||||
|
let marker = "default-file-logger-off-marker";
|
||||||
|
let subscriber = Registry::default().with(layers);
|
||||||
|
|
||||||
|
tracing::subscriber::with_default(subscriber, || {
|
||||||
|
tracing::info!(target: LOG_TARGET, "{}", marker);
|
||||||
|
std::thread::sleep(std::time::Duration::from_millis(300));
|
||||||
|
});
|
||||||
|
|
||||||
|
let content = std::fs::read_to_string(&log_path).unwrap_or_default();
|
||||||
|
assert!(
|
||||||
|
!content.contains(marker),
|
||||||
|
"default file logger level should filter info logs"
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
#[serial_test::serial]
|
||||||
|
fn file_logger_level_uses_env_filter_when_enabled() {
|
||||||
|
let _guard = EnvVarGuard::set(RUST_LOG, "debug");
|
||||||
|
let temp_dir = tempfile::tempdir().unwrap();
|
||||||
|
let log_file_name = "env-filter-test.log".to_string();
|
||||||
|
let log_path = temp_dir.path().join(&log_file_name);
|
||||||
|
|
||||||
|
let cfg = FileLoggerConfig {
|
||||||
|
level: Some(LevelFilter::INFO.to_string()),
|
||||||
|
file: Some(log_file_name),
|
||||||
|
dir: Some(temp_dir.path().to_string_lossy().to_string()),
|
||||||
|
..Default::default()
|
||||||
|
};
|
||||||
|
|
||||||
|
let (layers, _sender) = file_layers(cfg, true).unwrap();
|
||||||
|
let marker = "file-logger-env-filter-marker";
|
||||||
|
let subscriber = Registry::default().with(layers);
|
||||||
|
|
||||||
|
tracing::subscriber::with_default(subscriber, || {
|
||||||
|
tracing::debug!(target: LOG_TARGET, "{}", marker);
|
||||||
|
std::thread::sleep(std::time::Duration::from_millis(300));
|
||||||
|
});
|
||||||
|
|
||||||
|
let content = std::fs::read_to_string(&log_path).unwrap_or_default();
|
||||||
|
assert!(
|
||||||
|
content.contains(marker),
|
||||||
|
"enabled file logger should use RUST_LOG directives"
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
#[serial_test::serial]
|
||||||
|
fn file_logger_reload_uses_env_filter_when_enabled() {
|
||||||
|
let _guard = EnvVarGuard::set(RUST_LOG, "debug");
|
||||||
|
let temp_dir = tempfile::tempdir().unwrap();
|
||||||
|
let log_file_name = "reload-env-filter-test.log".to_string();
|
||||||
|
let log_path = temp_dir.path().join(&log_file_name);
|
||||||
|
|
||||||
|
let cfg = FileLoggerConfig {
|
||||||
|
file: Some(log_file_name),
|
||||||
|
dir: Some(temp_dir.path().to_string_lossy().to_string()),
|
||||||
|
..Default::default()
|
||||||
|
};
|
||||||
|
|
||||||
|
let (layers, sender) = file_layers(cfg, true).unwrap();
|
||||||
|
let sender = sender.expect("reload=true should return a sender");
|
||||||
|
let marker = "file-logger-reload-env-filter-marker";
|
||||||
|
let subscriber = Registry::default().with(layers);
|
||||||
|
|
||||||
|
tracing::subscriber::with_default(subscriber, || {
|
||||||
|
sender.send(LevelFilter::INFO.to_string()).unwrap();
|
||||||
|
std::thread::sleep(std::time::Duration::from_millis(300));
|
||||||
|
|
||||||
|
tracing::debug!(target: LOG_TARGET, "{}", marker);
|
||||||
|
std::thread::sleep(std::time::Duration::from_millis(300));
|
||||||
|
});
|
||||||
|
|
||||||
|
let content = std::fs::read_to_string(&log_path).unwrap_or_default();
|
||||||
|
assert!(
|
||||||
|
content.contains(marker),
|
||||||
|
"file logger enabled by reload should use RUST_LOG directives"
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
#[serial_test::serial]
|
||||||
|
fn file_logger_reload_off_ignores_env_filter() {
|
||||||
|
let _guard = EnvVarGuard::set(RUST_LOG, "info");
|
||||||
|
let temp_dir = tempfile::tempdir().unwrap();
|
||||||
|
let log_file_name = "reload-off-test.log".to_string();
|
||||||
|
let log_path = temp_dir.path().join(&log_file_name);
|
||||||
|
|
||||||
|
let cfg = FileLoggerConfig {
|
||||||
|
level: Some(LevelFilter::INFO.to_string()),
|
||||||
|
file: Some(log_file_name),
|
||||||
|
dir: Some(temp_dir.path().to_string_lossy().to_string()),
|
||||||
|
..Default::default()
|
||||||
|
};
|
||||||
|
|
||||||
|
let (layers, sender) = file_layers(cfg, true).unwrap();
|
||||||
|
let sender = sender.expect("reload=true should return a sender");
|
||||||
|
let marker = "file-logger-reload-off-marker";
|
||||||
|
let subscriber = Registry::default().with(layers);
|
||||||
|
|
||||||
|
tracing::subscriber::with_default(subscriber, || {
|
||||||
|
sender.send(LevelFilter::OFF.to_string()).unwrap();
|
||||||
|
std::thread::sleep(std::time::Duration::from_millis(300));
|
||||||
|
|
||||||
|
tracing::info!(target: LOG_TARGET, "{}", marker);
|
||||||
|
std::thread::sleep(std::time::Duration::from_millis(300));
|
||||||
|
});
|
||||||
|
|
||||||
|
let content = std::fs::read_to_string(&log_path).unwrap_or_default();
|
||||||
|
assert!(
|
||||||
|
!content.contains(marker),
|
||||||
|
"disabled file logger should ignore RUST_LOG directives"
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
#[serial_test::serial]
|
||||||
fn test_logger_reload() {
|
fn test_logger_reload() {
|
||||||
|
let _guard = EnvVarGuard::unset(RUST_LOG);
|
||||||
let temp_dir = tempfile::tempdir().unwrap();
|
let temp_dir = tempfile::tempdir().unwrap();
|
||||||
let log_file_name = "reload-test.log".to_string();
|
let log_file_name = "reload-test.log".to_string();
|
||||||
let log_path = temp_dir.path().join(&log_file_name);
|
let log_path = temp_dir.path().join(&log_file_name);
|
||||||
|
|||||||
@@ -74,7 +74,7 @@ use easytier::{
|
|||||||
common::{NatType, PortForwardConfigPb, SocketType},
|
common::{NatType, PortForwardConfigPb, SocketType},
|
||||||
peer_rpc::{GetGlobalPeerMapRequest, PeerCenterRpc, PeerCenterRpcClientFactory},
|
peer_rpc::{GetGlobalPeerMapRequest, PeerCenterRpc, PeerCenterRpcClientFactory},
|
||||||
rpc_impl::standalone::StandAloneClient,
|
rpc_impl::standalone::StandAloneClient,
|
||||||
rpc_types::controller::BaseController,
|
rpc_types::{controller::BaseController, error::Error as RpcError},
|
||||||
},
|
},
|
||||||
tunnel::{TunnelScheme, tcp::TcpTunnelConnector},
|
tunnel::{TunnelScheme, tcp::TcpTunnelConnector},
|
||||||
utils::{PeerRoutePair, string::cost_to_str},
|
utils::{PeerRoutePair, string::cost_to_str},
|
||||||
@@ -526,6 +526,40 @@ type LocalBoxFuture<'a, T> = Pin<Box<dyn Future<Output = Result<T, Error>> + 'a>
|
|||||||
type ForeignNetworkMap = BTreeMap<String, ForeignNetworkEntryPb>;
|
type ForeignNetworkMap = BTreeMap<String, ForeignNetworkEntryPb>;
|
||||||
type GlobalForeignNetworkMap = BTreeMap<u32, list_global_foreign_network_response::ForeignNetworks>;
|
type GlobalForeignNetworkMap = BTreeMap<u32, list_global_foreign_network_response::ForeignNetworks>;
|
||||||
|
|
||||||
|
fn is_missing_web_client_service(error: &RpcError) -> bool {
|
||||||
|
matches!(
|
||||||
|
error,
|
||||||
|
RpcError::InvalidServiceKey(service_name, _)
|
||||||
|
if service_name.trim_matches('"') == "WebClientService"
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
|
#[cfg(test)]
|
||||||
|
mod tests {
|
||||||
|
use super::*;
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn missing_web_client_service_matches_raw_service_name() {
|
||||||
|
let error = RpcError::InvalidServiceKey("WebClientService".to_string(), "".to_string());
|
||||||
|
|
||||||
|
assert!(is_missing_web_client_service(&error));
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn missing_web_client_service_matches_serialized_service_name() {
|
||||||
|
let error = RpcError::InvalidServiceKey("\"WebClientService\"".to_string(), "".to_string());
|
||||||
|
|
||||||
|
assert!(is_missing_web_client_service(&error));
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn missing_web_client_service_rejects_other_services() {
|
||||||
|
let error = RpcError::InvalidServiceKey("PeerManageRpc".to_string(), "".to_string());
|
||||||
|
|
||||||
|
assert!(!is_missing_web_client_service(&error));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
#[derive(serde::Serialize)]
|
#[derive(serde::Serialize)]
|
||||||
struct PeerListData {
|
struct PeerListData {
|
||||||
node_info: NodeInfo,
|
node_info: NodeInfo,
|
||||||
@@ -599,9 +633,15 @@ impl<'a> CommandHandler<'a> {
|
|||||||
}
|
}
|
||||||
|
|
||||||
let client = self.get_manage_client().await?;
|
let client = self.get_manage_client().await?;
|
||||||
let inst_ids = client
|
let list_response = match client
|
||||||
.list_network_instance(BaseController::default(), ListNetworkInstanceRequest {})
|
.list_network_instance(BaseController::default(), ListNetworkInstanceRequest {})
|
||||||
.await?
|
.await
|
||||||
|
{
|
||||||
|
Ok(response) => response,
|
||||||
|
Err(error) if is_missing_web_client_service(&error) => return Ok(None),
|
||||||
|
Err(error) => return Err(error.into()),
|
||||||
|
};
|
||||||
|
let inst_ids = list_response
|
||||||
.inst_ids
|
.inst_ids
|
||||||
.into_iter()
|
.into_iter()
|
||||||
.map(uuid::Uuid::from)
|
.map(uuid::Uuid::from)
|
||||||
|
|||||||
@@ -340,6 +340,11 @@ impl InstanceConfigPatcher {
|
|||||||
global_ctx.set_ipv6(Some(ipv6.into()));
|
global_ctx.set_ipv6(Some(ipv6.into()));
|
||||||
global_ctx.config.set_ipv6(Some(ipv6.into()));
|
global_ctx.config.set_ipv6(Some(ipv6.into()));
|
||||||
}
|
}
|
||||||
|
if let Some(disable_relay_data) = patch.disable_relay_data {
|
||||||
|
let mut flags = global_ctx.get_flags();
|
||||||
|
flags.disable_relay_data = disable_relay_data;
|
||||||
|
global_ctx.set_flags(flags);
|
||||||
|
}
|
||||||
if let Some(enabled) = patch.ipv6_public_addr_provider {
|
if let Some(enabled) = patch.ipv6_public_addr_provider {
|
||||||
global_ctx.config.set_ipv6_public_addr_provider(enabled);
|
global_ctx.config.set_ipv6_public_addr_provider(enabled);
|
||||||
provider_config_changed = true;
|
provider_config_changed = true;
|
||||||
|
|||||||
@@ -361,16 +361,8 @@ fn apply_public_ipv6_provider_runtime_state(
|
|||||||
let prefix_changed = global_ctx.set_advertised_ipv6_public_addr_prefix(next_prefix);
|
let prefix_changed = global_ctx.set_advertised_ipv6_public_addr_prefix(next_prefix);
|
||||||
|
|
||||||
let next_provider_enabled = matches!(state, PublicIpv6ProviderRuntimeState::Active(_));
|
let next_provider_enabled = matches!(state, PublicIpv6ProviderRuntimeState::Active(_));
|
||||||
let feature_changed = {
|
let feature_changed =
|
||||||
let mut feature_flags = global_ctx.get_feature_flags();
|
global_ctx.set_ipv6_public_addr_provider_feature_flag(next_provider_enabled);
|
||||||
if feature_flags.ipv6_public_addr_provider == next_provider_enabled {
|
|
||||||
false
|
|
||||||
} else {
|
|
||||||
feature_flags.ipv6_public_addr_provider = next_provider_enabled;
|
|
||||||
global_ctx.set_feature_flags(feature_flags);
|
|
||||||
true
|
|
||||||
}
|
|
||||||
};
|
|
||||||
|
|
||||||
prefix_changed || feature_changed
|
prefix_changed || feature_changed
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -816,6 +816,10 @@ impl NetworkConfig {
|
|||||||
flags.disable_upnp = disable_upnp;
|
flags.disable_upnp = disable_upnp;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if let Some(disable_relay_data) = self.disable_relay_data {
|
||||||
|
flags.disable_relay_data = disable_relay_data;
|
||||||
|
}
|
||||||
|
|
||||||
if let Some(disable_sym_hole_punching) = self.disable_sym_hole_punching {
|
if let Some(disable_sym_hole_punching) = self.disable_sym_hole_punching {
|
||||||
flags.disable_sym_hole_punching = disable_sym_hole_punching;
|
flags.disable_sym_hole_punching = disable_sym_hole_punching;
|
||||||
}
|
}
|
||||||
@@ -990,6 +994,7 @@ impl NetworkConfig {
|
|||||||
result.disable_tcp_hole_punching = Some(flags.disable_tcp_hole_punching);
|
result.disable_tcp_hole_punching = Some(flags.disable_tcp_hole_punching);
|
||||||
result.disable_udp_hole_punching = Some(flags.disable_udp_hole_punching);
|
result.disable_udp_hole_punching = Some(flags.disable_udp_hole_punching);
|
||||||
result.disable_upnp = Some(flags.disable_upnp);
|
result.disable_upnp = Some(flags.disable_upnp);
|
||||||
|
result.disable_relay_data = Some(flags.disable_relay_data);
|
||||||
result.disable_sym_hole_punching = Some(flags.disable_sym_hole_punching);
|
result.disable_sym_hole_punching = Some(flags.disable_sym_hole_punching);
|
||||||
result.enable_magic_dns = Some(flags.accept_dns);
|
result.enable_magic_dns = Some(flags.accept_dns);
|
||||||
result.mtu = Some(flags.mtu as i32);
|
result.mtu = Some(flags.mtu as i32);
|
||||||
|
|||||||
@@ -65,7 +65,7 @@ impl PeerCenterBase {
|
|||||||
return Err(Error::Shutdown);
|
return Err(Error::Shutdown);
|
||||||
};
|
};
|
||||||
rpc_mgr.rpc_server().registry().register(
|
rpc_mgr.rpc_server().registry().register(
|
||||||
PeerCenterRpcServer::new(PeerCenterServer::new(self.peer_mgr.my_peer_id())),
|
PeerCenterRpcServer::new(PeerCenterServer::new()),
|
||||||
&self.peer_mgr.get_global_ctx().get_network_name(),
|
&self.peer_mgr.get_global_ctx().get_network_name(),
|
||||||
);
|
);
|
||||||
Ok(())
|
Ok(())
|
||||||
@@ -486,7 +486,6 @@ impl PeerCenterPeerManagerTrait for PeerMapWithPeerRpcManager {
|
|||||||
#[cfg(test)]
|
#[cfg(test)]
|
||||||
mod tests {
|
mod tests {
|
||||||
use crate::{
|
use crate::{
|
||||||
peer_center::server::get_global_data,
|
|
||||||
peers::tests::{connect_peer_manager, create_mock_peer_manager, wait_route_appear},
|
peers::tests::{connect_peer_manager, create_mock_peer_manager, wait_route_appear},
|
||||||
tunnel::common::tests::wait_for_condition,
|
tunnel::common::tests::wait_for_condition,
|
||||||
};
|
};
|
||||||
@@ -515,25 +514,6 @@ mod tests {
|
|||||||
.await
|
.await
|
||||||
.unwrap();
|
.unwrap();
|
||||||
|
|
||||||
let center_peer = PeerCenterBase::select_center_peer(&peer_mgr_a)
|
|
||||||
.await
|
|
||||||
.unwrap();
|
|
||||||
let center_data = get_global_data(center_peer);
|
|
||||||
|
|
||||||
// wait center_data has 3 records for 10 seconds
|
|
||||||
wait_for_condition(
|
|
||||||
|| async {
|
|
||||||
if center_data.global_peer_map.len() == 4 {
|
|
||||||
println!("center data {:#?}", center_data.global_peer_map);
|
|
||||||
true
|
|
||||||
} else {
|
|
||||||
false
|
|
||||||
}
|
|
||||||
},
|
|
||||||
Duration::from_secs(20),
|
|
||||||
)
|
|
||||||
.await;
|
|
||||||
|
|
||||||
let mut digest = None;
|
let mut digest = None;
|
||||||
for pc in peer_centers.iter() {
|
for pc in peer_centers.iter() {
|
||||||
let rpc_service = pc.get_rpc_service();
|
let rpc_service = pc.get_rpc_service();
|
||||||
@@ -578,8 +558,5 @@ mod tests {
|
|||||||
route_cost.end_update();
|
route_cost.end_update();
|
||||||
assert!(!route_cost.need_update());
|
assert!(!route_cost.need_update());
|
||||||
}
|
}
|
||||||
|
|
||||||
let global_digest = get_global_data(center_peer).digest.load();
|
|
||||||
assert_eq!(digest.as_ref().unwrap(), &global_digest);
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -6,7 +6,6 @@ use std::{
|
|||||||
|
|
||||||
use crossbeam::atomic::AtomicCell;
|
use crossbeam::atomic::AtomicCell;
|
||||||
use dashmap::DashMap;
|
use dashmap::DashMap;
|
||||||
use once_cell::sync::Lazy;
|
|
||||||
use tokio::task::JoinSet;
|
use tokio::task::JoinSet;
|
||||||
|
|
||||||
use crate::{
|
use crate::{
|
||||||
@@ -35,50 +34,41 @@ pub(crate) struct PeerCenterInfoEntry {
|
|||||||
update_time: std::time::Instant,
|
update_time: std::time::Instant,
|
||||||
}
|
}
|
||||||
|
|
||||||
#[derive(Default)]
|
#[derive(Debug, Default)]
|
||||||
pub(crate) struct PeerCenterServerGlobalData {
|
struct PeerCenterServerData {
|
||||||
pub(crate) global_peer_map: DashMap<SrcDstPeerPair, PeerCenterInfoEntry>,
|
global_peer_map: DashMap<SrcDstPeerPair, PeerCenterInfoEntry>,
|
||||||
pub(crate) peer_report_time: DashMap<PeerId, std::time::Instant>,
|
peer_report_time: DashMap<PeerId, std::time::Instant>,
|
||||||
pub(crate) digest: AtomicCell<Digest>,
|
digest: AtomicCell<Digest>,
|
||||||
}
|
|
||||||
|
|
||||||
// a global unique instance for PeerCenterServer
|
|
||||||
pub(crate) static GLOBAL_DATA: Lazy<DashMap<PeerId, Arc<PeerCenterServerGlobalData>>> =
|
|
||||||
Lazy::new(DashMap::new);
|
|
||||||
|
|
||||||
pub(crate) fn get_global_data(node_id: PeerId) -> Arc<PeerCenterServerGlobalData> {
|
|
||||||
GLOBAL_DATA
|
|
||||||
.entry(node_id)
|
|
||||||
.or_insert_with(|| Arc::new(PeerCenterServerGlobalData::default()))
|
|
||||||
.value()
|
|
||||||
.clone()
|
|
||||||
}
|
}
|
||||||
|
|
||||||
#[derive(Clone, Debug)]
|
#[derive(Clone, Debug)]
|
||||||
pub struct PeerCenterServer {
|
pub struct PeerCenterServer {
|
||||||
// every peer has its own server, so use per-struct dash map is ok.
|
data: Arc<PeerCenterServerData>,
|
||||||
my_node_id: PeerId,
|
|
||||||
tasks: Arc<JoinSet<()>>,
|
tasks: Arc<JoinSet<()>>,
|
||||||
}
|
}
|
||||||
|
|
||||||
impl PeerCenterServer {
|
impl PeerCenterServer {
|
||||||
pub fn new(my_node_id: PeerId) -> Self {
|
pub fn new() -> Self {
|
||||||
|
let data = Arc::new(PeerCenterServerData::default());
|
||||||
|
let weak_data = Arc::downgrade(&data);
|
||||||
let mut tasks = JoinSet::new();
|
let mut tasks = JoinSet::new();
|
||||||
tasks.spawn(async move {
|
tasks.spawn(async move {
|
||||||
loop {
|
loop {
|
||||||
tokio::time::sleep(std::time::Duration::from_secs(10)).await;
|
tokio::time::sleep(std::time::Duration::from_secs(10)).await;
|
||||||
PeerCenterServer::clean_outdated_peer(my_node_id).await;
|
let Some(data) = weak_data.upgrade() else {
|
||||||
|
break;
|
||||||
|
};
|
||||||
|
PeerCenterServer::clean_outdated_peer_data(&data).await;
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
PeerCenterServer {
|
PeerCenterServer {
|
||||||
my_node_id,
|
data,
|
||||||
tasks: Arc::new(tasks),
|
tasks: Arc::new(tasks),
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
async fn clean_outdated_peer(my_node_id: PeerId) {
|
async fn clean_outdated_peer_data(data: &PeerCenterServerData) {
|
||||||
let data = get_global_data(my_node_id);
|
|
||||||
data.peer_report_time.retain(|_, v| {
|
data.peer_report_time.retain(|_, v| {
|
||||||
std::time::Instant::now().duration_since(*v) < std::time::Duration::from_secs(180)
|
std::time::Instant::now().duration_since(*v) < std::time::Duration::from_secs(180)
|
||||||
});
|
});
|
||||||
@@ -88,8 +78,7 @@ impl PeerCenterServer {
|
|||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
fn calc_global_digest(my_node_id: PeerId) -> Digest {
|
fn calc_global_digest_data(data: &PeerCenterServerData) -> Digest {
|
||||||
let data = get_global_data(my_node_id);
|
|
||||||
let mut hasher = std::collections::hash_map::DefaultHasher::new();
|
let mut hasher = std::collections::hash_map::DefaultHasher::new();
|
||||||
data.global_peer_map
|
data.global_peer_map
|
||||||
.iter()
|
.iter()
|
||||||
@@ -117,7 +106,7 @@ impl PeerCenterRpc for PeerCenterServer {
|
|||||||
|
|
||||||
tracing::debug!("receive report_peers");
|
tracing::debug!("receive report_peers");
|
||||||
|
|
||||||
let data = get_global_data(self.my_node_id);
|
let data = &self.data;
|
||||||
data.peer_report_time
|
data.peer_report_time
|
||||||
.insert(my_peer_id, std::time::Instant::now());
|
.insert(my_peer_id, std::time::Instant::now());
|
||||||
|
|
||||||
@@ -134,7 +123,7 @@ impl PeerCenterRpc for PeerCenterServer {
|
|||||||
}
|
}
|
||||||
|
|
||||||
data.digest
|
data.digest
|
||||||
.store(PeerCenterServer::calc_global_digest(self.my_node_id));
|
.store(PeerCenterServer::calc_global_digest_data(data));
|
||||||
|
|
||||||
Ok(ReportPeersResponse::default())
|
Ok(ReportPeersResponse::default())
|
||||||
}
|
}
|
||||||
@@ -147,7 +136,7 @@ impl PeerCenterRpc for PeerCenterServer {
|
|||||||
) -> Result<GetGlobalPeerMapResponse, rpc_types::error::Error> {
|
) -> Result<GetGlobalPeerMapResponse, rpc_types::error::Error> {
|
||||||
let digest = req.digest;
|
let digest = req.digest;
|
||||||
|
|
||||||
let data = get_global_data(self.my_node_id);
|
let data = &self.data;
|
||||||
if digest == data.digest.load() && digest != 0 {
|
if digest == data.digest.load() && digest != 0 {
|
||||||
return Ok(GetGlobalPeerMapResponse::default());
|
return Ok(GetGlobalPeerMapResponse::default());
|
||||||
}
|
}
|
||||||
@@ -171,3 +160,80 @@ impl PeerCenterRpc for PeerCenterServer {
|
|||||||
})
|
})
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[cfg(test)]
|
||||||
|
mod tests {
|
||||||
|
use super::*;
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn server_clones_share_instance_data() {
|
||||||
|
let server = PeerCenterServer::new();
|
||||||
|
let server_clone = server.clone();
|
||||||
|
|
||||||
|
let mut peers = PeerInfoForGlobalMap::default();
|
||||||
|
peers
|
||||||
|
.direct_peers
|
||||||
|
.insert(100, DirectConnectedPeerInfo { latency_ms: 3 });
|
||||||
|
|
||||||
|
server
|
||||||
|
.report_peers(
|
||||||
|
BaseController::default(),
|
||||||
|
ReportPeersRequest {
|
||||||
|
my_peer_id: 99,
|
||||||
|
peer_infos: Some(peers),
|
||||||
|
},
|
||||||
|
)
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
|
||||||
|
let resp = server_clone
|
||||||
|
.get_global_peer_map(
|
||||||
|
BaseController::default(),
|
||||||
|
GetGlobalPeerMapRequest { digest: 0 },
|
||||||
|
)
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
assert_eq!(1, resp.global_peer_map.len());
|
||||||
|
assert!(resp.global_peer_map[&99].direct_peers.contains_key(&100));
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn independent_server_instances_do_not_share_data() {
|
||||||
|
let server_a = PeerCenterServer::new();
|
||||||
|
let server_b = PeerCenterServer::new();
|
||||||
|
|
||||||
|
let mut peers = PeerInfoForGlobalMap::default();
|
||||||
|
peers
|
||||||
|
.direct_peers
|
||||||
|
.insert(101, DirectConnectedPeerInfo { latency_ms: 5 });
|
||||||
|
|
||||||
|
server_a
|
||||||
|
.report_peers(
|
||||||
|
BaseController::default(),
|
||||||
|
ReportPeersRequest {
|
||||||
|
my_peer_id: 100,
|
||||||
|
peer_infos: Some(peers),
|
||||||
|
},
|
||||||
|
)
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
|
||||||
|
let resp_a = server_a
|
||||||
|
.get_global_peer_map(
|
||||||
|
BaseController::default(),
|
||||||
|
GetGlobalPeerMapRequest { digest: 0 },
|
||||||
|
)
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
assert_eq!(1, resp_a.global_peer_map.len());
|
||||||
|
|
||||||
|
let resp_b = server_b
|
||||||
|
.get_global_peer_map(
|
||||||
|
BaseController::default(),
|
||||||
|
GetGlobalPeerMapRequest { digest: 0 },
|
||||||
|
)
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
assert!(resp_b.global_peer_map.is_empty());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
@@ -56,7 +56,7 @@ use super::{
|
|||||||
route_trait::NextHopPolicy,
|
route_trait::NextHopPolicy,
|
||||||
traffic_metrics::{
|
traffic_metrics::{
|
||||||
InstanceLabelKind, LogicalTrafficMetrics, TrafficKind, TrafficMetricRecorder,
|
InstanceLabelKind, LogicalTrafficMetrics, TrafficKind, TrafficMetricRecorder,
|
||||||
route_peer_info_instance_id, traffic_kind,
|
is_relay_data_packet_type, route_peer_info_instance_id, traffic_kind,
|
||||||
},
|
},
|
||||||
};
|
};
|
||||||
|
|
||||||
@@ -69,11 +69,16 @@ pub trait GlobalForeignNetworkAccessor: Send + Sync + 'static {
|
|||||||
struct ForeignNetworkEntry {
|
struct ForeignNetworkEntry {
|
||||||
my_peer_id: PeerId,
|
my_peer_id: PeerId,
|
||||||
|
|
||||||
|
// Node-global runtime flags, such as disable_relay_data, live on the parent
|
||||||
|
// context. The foreign context is scoped to the foreign network's OSPF view.
|
||||||
|
parent_global_ctx: ArcGlobalCtx,
|
||||||
global_ctx: ArcGlobalCtx,
|
global_ctx: ArcGlobalCtx,
|
||||||
network: NetworkIdentity,
|
network: NetworkIdentity,
|
||||||
peer_map: Arc<PeerMap>,
|
peer_map: Arc<PeerMap>,
|
||||||
relay_peer_map: Arc<RelayPeerMap>,
|
relay_peer_map: Arc<RelayPeerMap>,
|
||||||
peer_session_store: Arc<PeerSessionStore>,
|
peer_session_store: Arc<PeerSessionStore>,
|
||||||
|
// Static per-network permission from the whitelist check. disable_relay_data
|
||||||
|
// is the node-wide runtime override layered on top of this value.
|
||||||
relay_data: bool,
|
relay_data: bool,
|
||||||
pm_packet_sender: Mutex<Option<PacketRecvChan>>,
|
pm_packet_sender: Mutex<Option<PacketRecvChan>>,
|
||||||
|
|
||||||
@@ -82,7 +87,7 @@ struct ForeignNetworkEntry {
|
|||||||
|
|
||||||
packet_recv: Mutex<Option<PacketRecvChanReceiver>>,
|
packet_recv: Mutex<Option<PacketRecvChanReceiver>>,
|
||||||
|
|
||||||
bps_limiter: Arc<TokenBucket>,
|
bps_limiter: Option<Arc<TokenBucket>>,
|
||||||
|
|
||||||
peer_center: Arc<PeerCenterInstance>,
|
peer_center: Arc<PeerCenterInstance>,
|
||||||
|
|
||||||
@@ -186,14 +191,16 @@ impl ForeignNetworkEntry {
|
|||||||
);
|
);
|
||||||
|
|
||||||
let relay_bps_limit = global_ctx.config.get_flags().foreign_relay_bps_limit;
|
let relay_bps_limit = global_ctx.config.get_flags().foreign_relay_bps_limit;
|
||||||
|
let bps_limiter = (relay_bps_limit != u64::MAX).then(|| {
|
||||||
let limiter_config = LimiterConfig {
|
let limiter_config = LimiterConfig {
|
||||||
burst_rate: None,
|
burst_rate: None,
|
||||||
bps: Some(relay_bps_limit),
|
bps: Some(relay_bps_limit),
|
||||||
fill_duration_ms: None,
|
fill_duration_ms: None,
|
||||||
};
|
};
|
||||||
let bps_limiter = global_ctx
|
global_ctx
|
||||||
.token_bucket_manager()
|
.token_bucket_manager()
|
||||||
.get_or_create(&network.network_name, limiter_config.into());
|
.get_or_create(&network.network_name, limiter_config.into())
|
||||||
|
});
|
||||||
|
|
||||||
let peer_center = Arc::new(PeerCenterInstance::new(Arc::new(
|
let peer_center = Arc::new(PeerCenterInstance::new(Arc::new(
|
||||||
PeerMapWithPeerRpcManager {
|
PeerMapWithPeerRpcManager {
|
||||||
@@ -205,6 +212,7 @@ impl ForeignNetworkEntry {
|
|||||||
Self {
|
Self {
|
||||||
my_peer_id,
|
my_peer_id,
|
||||||
|
|
||||||
|
parent_global_ctx: global_ctx.clone(),
|
||||||
global_ctx: foreign_global_ctx,
|
global_ctx: foreign_global_ctx,
|
||||||
network,
|
network,
|
||||||
peer_map,
|
peer_map,
|
||||||
@@ -231,6 +239,27 @@ impl ForeignNetworkEntry {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn desired_avoid_relay_data_feature_flag(
|
||||||
|
parent_global_ctx: &ArcGlobalCtx,
|
||||||
|
relay_data: bool,
|
||||||
|
) -> bool {
|
||||||
|
!relay_data || parent_global_ctx.get_feature_flags().avoid_relay_data
|
||||||
|
}
|
||||||
|
|
||||||
|
fn sync_parent_relay_data_feature_flag(
|
||||||
|
parent_global_ctx: &ArcGlobalCtx,
|
||||||
|
global_ctx: &ArcGlobalCtx,
|
||||||
|
relay_data: bool,
|
||||||
|
) -> bool {
|
||||||
|
let avoid_relay_data =
|
||||||
|
Self::desired_avoid_relay_data_feature_flag(parent_global_ctx, relay_data);
|
||||||
|
if global_ctx.get_feature_flags().avoid_relay_data == avoid_relay_data {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
global_ctx.set_avoid_relay_data_preference(avoid_relay_data)
|
||||||
|
}
|
||||||
|
|
||||||
fn build_foreign_global_ctx(
|
fn build_foreign_global_ctx(
|
||||||
network: &NetworkIdentity,
|
network: &NetworkIdentity,
|
||||||
global_ctx: ArcGlobalCtx,
|
global_ctx: ArcGlobalCtx,
|
||||||
@@ -258,10 +287,9 @@ impl ForeignNetworkEntry {
|
|||||||
|
|
||||||
let mut feature_flag = global_ctx.get_feature_flags();
|
let mut feature_flag = global_ctx.get_feature_flags();
|
||||||
feature_flag.is_public_server = true;
|
feature_flag.is_public_server = true;
|
||||||
if !relay_data {
|
feature_flag.avoid_relay_data =
|
||||||
feature_flag.avoid_relay_data = true;
|
Self::desired_avoid_relay_data_feature_flag(&global_ctx, relay_data);
|
||||||
}
|
foreign_global_ctx.set_base_advertised_feature_flags(feature_flag);
|
||||||
foreign_global_ctx.set_feature_flags(feature_flag);
|
|
||||||
|
|
||||||
for u in global_ctx.get_running_listeners().into_iter() {
|
for u in global_ctx.get_running_listeners().into_iter() {
|
||||||
foreign_global_ctx.add_running_listener(u);
|
foreign_global_ctx.add_running_listener(u);
|
||||||
@@ -412,6 +440,7 @@ impl ForeignNetworkEntry {
|
|||||||
let peer_map = self.peer_map.clone();
|
let peer_map = self.peer_map.clone();
|
||||||
let relay_peer_map = self.relay_peer_map.clone();
|
let relay_peer_map = self.relay_peer_map.clone();
|
||||||
let traffic_metrics = self.traffic_metrics.clone();
|
let traffic_metrics = self.traffic_metrics.clone();
|
||||||
|
let parent_global_ctx = self.parent_global_ctx.clone();
|
||||||
let relay_data = self.relay_data;
|
let relay_data = self.relay_data;
|
||||||
let pm_sender = self.pm_packet_sender.lock().await.take().unwrap();
|
let pm_sender = self.pm_packet_sender.lock().await.take().unwrap();
|
||||||
let network_name = self.network.network_name.clone();
|
let network_name = self.network.network_name.clone();
|
||||||
@@ -497,14 +526,21 @@ impl ForeignNetworkEntry {
|
|||||||
"ignore packet in foreign network"
|
"ignore packet in foreign network"
|
||||||
);
|
);
|
||||||
} else {
|
} else {
|
||||||
if packet_type == PacketType::Data as u8
|
if is_relay_data_packet_type(packet_type) {
|
||||||
|| packet_type == PacketType::KcpSrc as u8
|
let disable_relay_data = parent_global_ctx.flags_arc().disable_relay_data;
|
||||||
|| packet_type == PacketType::KcpDst as u8
|
if !relay_data || disable_relay_data {
|
||||||
{
|
tracing::debug!(
|
||||||
if !relay_data {
|
?from_peer_id,
|
||||||
|
?to_peer_id,
|
||||||
|
packet_type,
|
||||||
|
disable_relay_data,
|
||||||
|
"drop foreign network relay data"
|
||||||
|
);
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
if !bps_limiter.try_consume(len.into()) {
|
if let Some(bps_limiter) = bps_limiter.as_ref()
|
||||||
|
&& !bps_limiter.try_consume(len.into())
|
||||||
|
{
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -589,10 +625,31 @@ impl ForeignNetworkEntry {
|
|||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
async fn run_parent_feature_flag_sync_routine(&self) {
|
||||||
|
let parent_global_ctx = self.parent_global_ctx.clone();
|
||||||
|
let global_ctx = self.global_ctx.clone();
|
||||||
|
let relay_data = self.relay_data;
|
||||||
|
self.tasks.lock().await.spawn(async move {
|
||||||
|
let mut parent_events = parent_global_ctx.subscribe();
|
||||||
|
loop {
|
||||||
|
ForeignNetworkEntry::sync_parent_relay_data_feature_flag(
|
||||||
|
&parent_global_ctx,
|
||||||
|
&global_ctx,
|
||||||
|
relay_data,
|
||||||
|
);
|
||||||
|
|
||||||
|
if parent_events.recv().await.is_err() {
|
||||||
|
parent_events = parent_global_ctx.subscribe();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
async fn prepare(&self, accessor: Box<dyn GlobalForeignNetworkAccessor>) {
|
async fn prepare(&self, accessor: Box<dyn GlobalForeignNetworkAccessor>) {
|
||||||
self.prepare_route(accessor).await;
|
self.prepare_route(accessor).await;
|
||||||
self.start_packet_recv().await;
|
self.start_packet_recv().await;
|
||||||
self.run_relay_session_gc_routine().await;
|
self.run_relay_session_gc_routine().await;
|
||||||
|
self.run_parent_feature_flag_sync_routine().await;
|
||||||
self.peer_rpc.run();
|
self.peer_rpc.run();
|
||||||
self.peer_center.init().await;
|
self.peer_center.init().await;
|
||||||
}
|
}
|
||||||
@@ -660,6 +717,7 @@ impl ForeignNetworkManagerData {
|
|||||||
fn remove_network(&self, network_name: &String) {
|
fn remove_network(&self, network_name: &String) {
|
||||||
let _l = self.lock.lock().unwrap();
|
let _l = self.lock.lock().unwrap();
|
||||||
if let Some(old) = self.network_peer_maps.remove(network_name) {
|
if let Some(old) = self.network_peer_maps.remove(network_name) {
|
||||||
|
old.1.traffic_metrics.clear_peer_cache();
|
||||||
let to_remove_peers = old.1.peer_map.list_peers();
|
let to_remove_peers = old.1.peer_map.list_peers();
|
||||||
for p in to_remove_peers {
|
for p in to_remove_peers {
|
||||||
self.peer_network_map.remove_if(&p, |_, v| {
|
self.peer_network_map.remove_if(&p, |_, v| {
|
||||||
@@ -669,6 +727,9 @@ impl ForeignNetworkManagerData {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
self.network_peer_last_update.remove(network_name);
|
self.network_peer_last_update.remove(network_name);
|
||||||
|
shrink_dashmap(&self.peer_network_map, None);
|
||||||
|
shrink_dashmap(&self.network_peer_maps, None);
|
||||||
|
shrink_dashmap(&self.network_peer_last_update, None);
|
||||||
}
|
}
|
||||||
|
|
||||||
#[allow(clippy::too_many_arguments)]
|
#[allow(clippy::too_many_arguments)]
|
||||||
@@ -941,12 +1002,14 @@ impl ForeignNetworkManager {
|
|||||||
async fn start_event_handler(&self, entry: &ForeignNetworkEntry) {
|
async fn start_event_handler(&self, entry: &ForeignNetworkEntry) {
|
||||||
let data = self.data.clone();
|
let data = self.data.clone();
|
||||||
let network_name = entry.network.network_name.clone();
|
let network_name = entry.network.network_name.clone();
|
||||||
|
let traffic_metrics = entry.traffic_metrics.clone();
|
||||||
let mut s = entry.global_ctx.subscribe();
|
let mut s = entry.global_ctx.subscribe();
|
||||||
self.tasks.lock().unwrap().spawn(async move {
|
self.tasks.lock().unwrap().spawn(async move {
|
||||||
while let Ok(e) = s.recv().await {
|
while let Ok(e) = s.recv().await {
|
||||||
match &e {
|
match &e {
|
||||||
GlobalCtxEvent::PeerRemoved(peer_id) => {
|
GlobalCtxEvent::PeerRemoved(peer_id) => {
|
||||||
tracing::info!(?e, "remove peer from foreign network manager");
|
tracing::info!(?e, "remove peer from foreign network manager");
|
||||||
|
traffic_metrics.remove_peer(*peer_id);
|
||||||
data.remove_peer(*peer_id, &network_name);
|
data.remove_peer(*peer_id, &network_name);
|
||||||
data.network_peer_last_update
|
data.network_peer_last_update
|
||||||
.insert(network_name.clone(), SystemTime::now());
|
.insert(network_name.clone(), SystemTime::now());
|
||||||
@@ -965,6 +1028,7 @@ impl ForeignNetworkManager {
|
|||||||
}
|
}
|
||||||
// if lagged or recv done just remove the network
|
// if lagged or recv done just remove the network
|
||||||
tracing::error!("global event handler at foreign network manager exit");
|
tracing::error!("global event handler at foreign network manager exit");
|
||||||
|
traffic_metrics.clear_peer_cache();
|
||||||
data.remove_network(&network_name);
|
data.remove_network(&network_name);
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
@@ -1397,6 +1461,92 @@ pub mod tests {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn disable_relay_data_blocks_foreign_network_transit_data() {
|
||||||
|
let pm_center = create_mock_peer_manager_with_mock_stun(NatType::Unknown).await;
|
||||||
|
let pma_net1 = create_mock_peer_manager_for_foreign_network("net1").await;
|
||||||
|
let pmb_net1 = create_mock_peer_manager_for_foreign_network("net1").await;
|
||||||
|
|
||||||
|
connect_peer_manager(pma_net1.clone(), pm_center.clone()).await;
|
||||||
|
connect_peer_manager(pmb_net1.clone(), pm_center.clone()).await;
|
||||||
|
wait_route_appear(pma_net1.clone(), pmb_net1.clone())
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
|
||||||
|
let mut flags = pm_center.get_global_ctx().get_flags();
|
||||||
|
flags.disable_relay_data = true;
|
||||||
|
pm_center.get_global_ctx().set_flags(flags);
|
||||||
|
pm_center
|
||||||
|
.get_global_ctx()
|
||||||
|
.issue_event(GlobalCtxEvent::ConfigPatched(Default::default()));
|
||||||
|
|
||||||
|
let center_peer_id = pm_center
|
||||||
|
.get_foreign_network_manager()
|
||||||
|
.get_network_peer_id("net1")
|
||||||
|
.unwrap();
|
||||||
|
wait_for_condition(
|
||||||
|
|| {
|
||||||
|
let pma_net1 = pma_net1.clone();
|
||||||
|
async move {
|
||||||
|
pma_net1.list_routes().await.iter().any(|route| {
|
||||||
|
route.peer_id == center_peer_id
|
||||||
|
&& route
|
||||||
|
.feature_flag
|
||||||
|
.as_ref()
|
||||||
|
.map(|flag| flag.avoid_relay_data)
|
||||||
|
.unwrap_or(false)
|
||||||
|
})
|
||||||
|
}
|
||||||
|
},
|
||||||
|
Duration::from_secs(5),
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
|
||||||
|
let network_labels =
|
||||||
|
LabelSet::new().with_label_type(LabelType::NetworkName("net1".to_string()));
|
||||||
|
let forwarded_bytes_before = metric_value(
|
||||||
|
&pm_center,
|
||||||
|
MetricName::TrafficBytesForwarded,
|
||||||
|
network_labels.clone(),
|
||||||
|
);
|
||||||
|
let forwarded_packets_before = metric_value(
|
||||||
|
&pm_center,
|
||||||
|
MetricName::TrafficPacketsForwarded,
|
||||||
|
network_labels.clone(),
|
||||||
|
);
|
||||||
|
|
||||||
|
let mut transit_pkt = ZCPacket::new_with_payload(b"foreign-transit-disabled");
|
||||||
|
transit_pkt.fill_peer_manager_hdr(
|
||||||
|
pma_net1.my_peer_id(),
|
||||||
|
pmb_net1.my_peer_id(),
|
||||||
|
PacketType::Data as u8,
|
||||||
|
);
|
||||||
|
pma_net1
|
||||||
|
.get_foreign_network_client()
|
||||||
|
.send_msg(transit_pkt, center_peer_id)
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
|
||||||
|
tokio::time::sleep(Duration::from_millis(300)).await;
|
||||||
|
|
||||||
|
assert_eq!(
|
||||||
|
metric_value(
|
||||||
|
&pm_center,
|
||||||
|
MetricName::TrafficBytesForwarded,
|
||||||
|
network_labels.clone()
|
||||||
|
),
|
||||||
|
forwarded_bytes_before
|
||||||
|
);
|
||||||
|
assert_eq!(
|
||||||
|
metric_value(
|
||||||
|
&pm_center,
|
||||||
|
MetricName::TrafficPacketsForwarded,
|
||||||
|
network_labels
|
||||||
|
),
|
||||||
|
forwarded_packets_before
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
async fn foreign_network_transit_control_forwarding_records_control_forwarded_metrics() {
|
async fn foreign_network_transit_control_forwarding_records_control_forwarded_metrics() {
|
||||||
let pm_center = create_mock_peer_manager_with_mock_stun(NatType::Unknown).await;
|
let pm_center = create_mock_peer_manager_with_mock_stun(NatType::Unknown).await;
|
||||||
@@ -1409,6 +1559,10 @@ pub mod tests {
|
|||||||
.await
|
.await
|
||||||
.unwrap();
|
.unwrap();
|
||||||
|
|
||||||
|
let mut flags = pm_center.get_global_ctx().get_flags();
|
||||||
|
flags.disable_relay_data = true;
|
||||||
|
pm_center.get_global_ctx().set_flags(flags);
|
||||||
|
|
||||||
let center_peer_id = pm_center
|
let center_peer_id = pm_center
|
||||||
.get_foreign_network_manager()
|
.get_foreign_network_manager()
|
||||||
.get_network_peer_id("net1")
|
.get_network_peer_id("net1")
|
||||||
@@ -1461,6 +1615,58 @@ pub mod tests {
|
|||||||
.await;
|
.await;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn foreign_network_peer_removed_clears_traffic_metric_peer_cache() {
|
||||||
|
let pm_center = create_mock_peer_manager_with_mock_stun(NatType::Unknown).await;
|
||||||
|
let pma_net1 = create_mock_peer_manager_for_foreign_network("net1").await;
|
||||||
|
|
||||||
|
connect_peer_manager(pma_net1.clone(), pm_center.clone()).await;
|
||||||
|
wait_for_condition(
|
||||||
|
|| {
|
||||||
|
let pm_center = pm_center.clone();
|
||||||
|
async move {
|
||||||
|
pm_center
|
||||||
|
.get_foreign_network_manager()
|
||||||
|
.get_network_peer_id("net1")
|
||||||
|
.is_some()
|
||||||
|
}
|
||||||
|
},
|
||||||
|
Duration::from_secs(5),
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
|
||||||
|
let entry = pm_center
|
||||||
|
.get_foreign_network_manager()
|
||||||
|
.data
|
||||||
|
.get_network_entry("net1")
|
||||||
|
.unwrap();
|
||||||
|
|
||||||
|
entry
|
||||||
|
.traffic_metrics
|
||||||
|
.record_rx(pma_net1.my_peer_id(), PacketType::Data as u8, 128)
|
||||||
|
.await;
|
||||||
|
|
||||||
|
assert!(
|
||||||
|
entry
|
||||||
|
.traffic_metrics
|
||||||
|
.contains_peer_cache(pma_net1.my_peer_id())
|
||||||
|
);
|
||||||
|
|
||||||
|
entry
|
||||||
|
.global_ctx
|
||||||
|
.issue_event(GlobalCtxEvent::PeerRemoved(pma_net1.my_peer_id()));
|
||||||
|
|
||||||
|
wait_for_condition(
|
||||||
|
|| {
|
||||||
|
let entry = entry.clone();
|
||||||
|
let peer_id = pma_net1.my_peer_id();
|
||||||
|
async move { !entry.traffic_metrics.contains_peer_cache(peer_id) }
|
||||||
|
},
|
||||||
|
Duration::from_secs(5),
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
async fn foreign_network_encapsulated_forwarding_records_tx_metrics() {
|
async fn foreign_network_encapsulated_forwarding_records_tx_metrics() {
|
||||||
set_global_var!(OSPF_UPDATE_MY_GLOBAL_FOREIGN_NETWORK_INTERVAL_SEC, 1);
|
set_global_var!(OSPF_UPDATE_MY_GLOBAL_FOREIGN_NETWORK_INTERVAL_SEC, 1);
|
||||||
@@ -1657,6 +1863,81 @@ pub mod tests {
|
|||||||
));
|
));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn foreign_entry_feature_flag_tracks_parent_disable_relay_data_toggle() {
|
||||||
|
let global_ctx = get_mock_global_ctx_with_network(Some(NetworkIdentity::new(
|
||||||
|
"__access__".to_string(),
|
||||||
|
"access_secret".to_string(),
|
||||||
|
)));
|
||||||
|
let foreign_network = NetworkIdentity::new("net1".to_string(), "net1_secret".to_string());
|
||||||
|
let (pm_packet_sender, _pm_packet_recv) = create_packet_recv_chan();
|
||||||
|
let entry = ForeignNetworkEntry::new(
|
||||||
|
foreign_network,
|
||||||
|
1,
|
||||||
|
global_ctx.clone(),
|
||||||
|
true,
|
||||||
|
Arc::new(PeerSessionStore::new()),
|
||||||
|
pm_packet_sender,
|
||||||
|
);
|
||||||
|
assert!(!entry.global_ctx.get_feature_flags().avoid_relay_data);
|
||||||
|
|
||||||
|
entry.run_parent_feature_flag_sync_routine().await;
|
||||||
|
|
||||||
|
let mut flags = global_ctx.get_flags();
|
||||||
|
flags.disable_relay_data = true;
|
||||||
|
global_ctx.set_flags(flags);
|
||||||
|
global_ctx.issue_event(GlobalCtxEvent::ConfigPatched(Default::default()));
|
||||||
|
|
||||||
|
wait_for_condition(
|
||||||
|
|| async { entry.global_ctx.get_feature_flags().avoid_relay_data },
|
||||||
|
Duration::from_secs(2),
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
|
||||||
|
let mut flags = global_ctx.get_flags();
|
||||||
|
flags.disable_relay_data = false;
|
||||||
|
global_ctx.set_flags(flags);
|
||||||
|
global_ctx.issue_event(GlobalCtxEvent::ConfigPatched(Default::default()));
|
||||||
|
|
||||||
|
wait_for_condition(
|
||||||
|
|| async { !entry.global_ctx.get_feature_flags().avoid_relay_data },
|
||||||
|
Duration::from_secs(2),
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn foreign_entry_without_relay_data_keeps_avoid_feature_flag() {
|
||||||
|
let global_ctx = get_mock_global_ctx_with_network(Some(NetworkIdentity::new(
|
||||||
|
"__access__".to_string(),
|
||||||
|
"access_secret".to_string(),
|
||||||
|
)));
|
||||||
|
let foreign_network = NetworkIdentity::new("net1".to_string(), "net1_secret".to_string());
|
||||||
|
let (pm_packet_sender, _pm_packet_recv) = create_packet_recv_chan();
|
||||||
|
let entry = ForeignNetworkEntry::new(
|
||||||
|
foreign_network,
|
||||||
|
1,
|
||||||
|
global_ctx.clone(),
|
||||||
|
false,
|
||||||
|
Arc::new(PeerSessionStore::new()),
|
||||||
|
pm_packet_sender,
|
||||||
|
);
|
||||||
|
|
||||||
|
assert!(entry.global_ctx.get_feature_flags().avoid_relay_data);
|
||||||
|
|
||||||
|
let mut flags = global_ctx.get_flags();
|
||||||
|
flags.disable_relay_data = false;
|
||||||
|
global_ctx.set_flags(flags);
|
||||||
|
|
||||||
|
ForeignNetworkEntry::sync_parent_relay_data_feature_flag(
|
||||||
|
&global_ctx,
|
||||||
|
&entry.global_ctx,
|
||||||
|
entry.relay_data,
|
||||||
|
);
|
||||||
|
|
||||||
|
assert!(entry.global_ctx.get_feature_flags().avoid_relay_data);
|
||||||
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
fn credential_trust_path_rejects_admin_identity() {
|
fn credential_trust_path_rejects_admin_identity() {
|
||||||
assert!(ForeignNetworkManager::should_reject_credential_trust_path(
|
assert!(ForeignNetworkManager::should_reject_credential_trust_path(
|
||||||
|
|||||||
@@ -1352,7 +1352,9 @@ impl PeerConn {
|
|||||||
|
|
||||||
let is_foreign_network = conn_info_for_instrument.network_name
|
let is_foreign_network = conn_info_for_instrument.network_name
|
||||||
!= self.global_ctx.get_network_identity().network_name;
|
!= self.global_ctx.get_network_identity().network_name;
|
||||||
let recv_limiter = if is_foreign_network {
|
let recv_limiter = if is_foreign_network
|
||||||
|
&& self.global_ctx.get_flags().foreign_relay_bps_limit != u64::MAX
|
||||||
|
{
|
||||||
let relay_network_bps_limit = self.global_ctx.get_flags().foreign_relay_bps_limit;
|
let relay_network_bps_limit = self.global_ctx.get_flags().foreign_relay_bps_limit;
|
||||||
let limiter_config = LimiterConfig {
|
let limiter_config = LimiterConfig {
|
||||||
burst_rate: None,
|
burst_rate: None,
|
||||||
|
|||||||
@@ -38,7 +38,7 @@ use crate::{
|
|||||||
route_trait::{ForeignNetworkRouteInfoMap, MockRoute, NextHopPolicy, RouteInterface},
|
route_trait::{ForeignNetworkRouteInfoMap, MockRoute, NextHopPolicy, RouteInterface},
|
||||||
traffic_metrics::{
|
traffic_metrics::{
|
||||||
InstanceLabelKind, LogicalTrafficMetrics, TrafficKind, TrafficMetricRecorder,
|
InstanceLabelKind, LogicalTrafficMetrics, TrafficKind, TrafficMetricRecorder,
|
||||||
route_peer_info_instance_id, traffic_kind,
|
is_relay_data_packet_type, route_peer_info_instance_id, traffic_kind,
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
proto::{
|
proto::{
|
||||||
@@ -263,9 +263,7 @@ impl PeerManager {
|
|||||||
.is_err()
|
.is_err()
|
||||||
{
|
{
|
||||||
// if local network is not in whitelist, avoid relay data when exist any other route path
|
// if local network is not in whitelist, avoid relay data when exist any other route path
|
||||||
let mut f = global_ctx.get_feature_flags();
|
global_ctx.set_avoid_relay_data_preference(true);
|
||||||
f.avoid_relay_data = true;
|
|
||||||
global_ctx.set_feature_flags(f);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
let is_secure_mode_enabled = global_ctx
|
let is_secure_mode_enabled = global_ctx
|
||||||
@@ -689,6 +687,11 @@ impl PeerManager {
|
|||||||
Ok(())
|
Ok(())
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn release_reserved_peer_id(&self, network_name: &str) {
|
||||||
|
self.reserved_my_peer_id_map.remove(network_name);
|
||||||
|
shrink_dashmap(&self.reserved_my_peer_id_map, None);
|
||||||
|
}
|
||||||
|
|
||||||
#[tracing::instrument(ret)]
|
#[tracing::instrument(ret)]
|
||||||
pub async fn add_tunnel_as_server(
|
pub async fn add_tunnel_as_server(
|
||||||
&self,
|
&self,
|
||||||
@@ -704,7 +707,8 @@ impl PeerManager {
|
|||||||
tunnel,
|
tunnel,
|
||||||
self.peer_session_store.clone(),
|
self.peer_session_store.clone(),
|
||||||
);
|
);
|
||||||
conn.do_handshake_as_server_ext(|peer, network_name:&str| {
|
let mut reserved_peer_id_network_name = None;
|
||||||
|
let handshake_ret = conn.do_handshake_as_server_ext(|peer, network_name:&str| {
|
||||||
if network_name
|
if network_name
|
||||||
== self.global_ctx.get_network_identity().network_name
|
== self.global_ctx.get_network_identity().network_name
|
||||||
{
|
{
|
||||||
@@ -715,6 +719,7 @@ impl PeerManager {
|
|||||||
.foreign_network_manager
|
.foreign_network_manager
|
||||||
.get_network_peer_id(network_name);
|
.get_network_peer_id(network_name);
|
||||||
if peer_id.is_none() {
|
if peer_id.is_none() {
|
||||||
|
reserved_peer_id_network_name = Some(network_name.to_string());
|
||||||
peer_id = Some(*self.reserved_my_peer_id_map.entry(network_name.to_string()).or_insert_with(|| {
|
peer_id = Some(*self.reserved_my_peer_id_map.entry(network_name.to_string()).or_insert_with(|| {
|
||||||
rand::random::<PeerId>()
|
rand::random::<PeerId>()
|
||||||
}).value());
|
}).value());
|
||||||
@@ -730,7 +735,14 @@ impl PeerManager {
|
|||||||
|
|
||||||
Ok(())
|
Ok(())
|
||||||
})
|
})
|
||||||
.await?;
|
.await;
|
||||||
|
|
||||||
|
if let Err(err) = handshake_ret {
|
||||||
|
if let Some(network_name) = reserved_peer_id_network_name {
|
||||||
|
self.release_reserved_peer_id(&network_name);
|
||||||
|
}
|
||||||
|
return Err(err);
|
||||||
|
}
|
||||||
|
|
||||||
let peer_identity = conn.get_network_identity();
|
let peer_identity = conn.get_network_identity();
|
||||||
let peer_network_name = peer_identity.network_name.clone();
|
let peer_network_name = peer_identity.network_name.clone();
|
||||||
@@ -749,6 +761,7 @@ impl PeerManager {
|
|||||||
|
|
||||||
if !is_local_network && self.global_ctx.get_flags().private_mode && !foreign_network_allowed
|
if !is_local_network && self.global_ctx.get_flags().private_mode && !foreign_network_allowed
|
||||||
{
|
{
|
||||||
|
self.release_reserved_peer_id(&peer_network_name);
|
||||||
return Err(Error::SecretKeyError(
|
return Err(Error::SecretKeyError(
|
||||||
"private mode is turned on, foreign network secret mismatch".to_string(),
|
"private mode is turned on, foreign network secret mismatch".to_string(),
|
||||||
));
|
));
|
||||||
@@ -756,14 +769,18 @@ impl PeerManager {
|
|||||||
|
|
||||||
conn.set_is_hole_punched(!is_directly_connected);
|
conn.set_is_hole_punched(!is_directly_connected);
|
||||||
|
|
||||||
if is_local_network {
|
let add_peer_ret = if is_local_network {
|
||||||
self.add_new_peer_conn(conn).await?;
|
self.add_new_peer_conn(conn).await
|
||||||
} else {
|
} else {
|
||||||
self.foreign_network_manager.add_peer_conn(conn).await?;
|
self.foreign_network_manager.add_peer_conn(conn).await
|
||||||
|
};
|
||||||
|
|
||||||
|
if let Err(err) = add_peer_ret {
|
||||||
|
self.release_reserved_peer_id(&peer_network_name);
|
||||||
|
return Err(err);
|
||||||
}
|
}
|
||||||
|
|
||||||
self.reserved_my_peer_id_map.remove(&peer_network_name);
|
self.release_reserved_peer_id(&peer_network_name);
|
||||||
shrink_dashmap(&self.reserved_my_peer_id_map, None);
|
|
||||||
|
|
||||||
tracing::info!("add tunnel as server done");
|
tracing::info!("add tunnel as server done");
|
||||||
Ok(())
|
Ok(())
|
||||||
@@ -774,6 +791,7 @@ impl PeerManager {
|
|||||||
my_peer_id: PeerId,
|
my_peer_id: PeerId,
|
||||||
peer_map: &PeerMap,
|
peer_map: &PeerMap,
|
||||||
foreign_network_mgr: &ForeignNetworkManager,
|
foreign_network_mgr: &ForeignNetworkManager,
|
||||||
|
disable_relay_data: bool,
|
||||||
) -> Result<(), ZCPacket> {
|
) -> Result<(), ZCPacket> {
|
||||||
let pm_header = packet.peer_manager_header().unwrap();
|
let pm_header = packet.peer_manager_header().unwrap();
|
||||||
if pm_header.packet_type != PacketType::ForeignNetworkPacket as u8 {
|
if pm_header.packet_type != PacketType::ForeignNetworkPacket as u8 {
|
||||||
@@ -783,6 +801,16 @@ impl PeerManager {
|
|||||||
let from_peer_id = pm_header.from_peer_id.get();
|
let from_peer_id = pm_header.from_peer_id.get();
|
||||||
let to_peer_id = pm_header.to_peer_id.get();
|
let to_peer_id = pm_header.to_peer_id.get();
|
||||||
|
|
||||||
|
if disable_relay_data && Self::is_relay_data_zc_packet(&packet) {
|
||||||
|
tracing::debug!(
|
||||||
|
?from_peer_id,
|
||||||
|
?to_peer_id,
|
||||||
|
inner_packet_type = ?packet.foreign_network_inner_packet_type(),
|
||||||
|
"drop foreign network relay data while relay data is disabled"
|
||||||
|
);
|
||||||
|
return Ok(());
|
||||||
|
}
|
||||||
|
|
||||||
let foreign_hdr = packet.foreign_network_hdr().unwrap();
|
let foreign_hdr = packet.foreign_network_hdr().unwrap();
|
||||||
let foreign_network_name = foreign_hdr.get_network_name(packet.payload());
|
let foreign_network_name = foreign_hdr.get_network_name(packet.payload());
|
||||||
let foreign_peer_id = foreign_hdr.get_dst_peer_id();
|
let foreign_peer_id = foreign_hdr.get_dst_peer_id();
|
||||||
@@ -872,6 +900,29 @@ impl PeerManager {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn is_relay_data_packet(packet_type: u8) -> bool {
|
||||||
|
is_relay_data_packet_type(packet_type)
|
||||||
|
}
|
||||||
|
|
||||||
|
fn is_relay_data_zc_packet(packet: &ZCPacket) -> bool {
|
||||||
|
let Some(hdr) = packet.peer_manager_header() else {
|
||||||
|
return false;
|
||||||
|
};
|
||||||
|
|
||||||
|
if hdr.packet_type == PacketType::ForeignNetworkPacket as u8 {
|
||||||
|
let inner_packet_type = packet.foreign_network_inner_packet_type();
|
||||||
|
if inner_packet_type.is_none() {
|
||||||
|
tracing::warn!(
|
||||||
|
?hdr,
|
||||||
|
"foreign network packet has unparseable inner peer manager header"
|
||||||
|
);
|
||||||
|
}
|
||||||
|
return inner_packet_type.is_none_or(Self::is_relay_data_packet);
|
||||||
|
}
|
||||||
|
|
||||||
|
Self::is_relay_data_packet(hdr.packet_type)
|
||||||
|
}
|
||||||
|
|
||||||
async fn start_peer_recv(&self) {
|
async fn start_peer_recv(&self) {
|
||||||
let mut recv = self.packet_recv.lock().await.take().unwrap();
|
let mut recv = self.packet_recv.lock().await.take().unwrap();
|
||||||
let my_peer_id = self.my_peer_id;
|
let my_peer_id = self.my_peer_id;
|
||||||
@@ -925,14 +976,21 @@ impl PeerManager {
|
|||||||
self.tasks.lock().await.spawn(async move {
|
self.tasks.lock().await.spawn(async move {
|
||||||
tracing::trace!("start_peer_recv");
|
tracing::trace!("start_peer_recv");
|
||||||
while let Ok(ret) = recv_packet_from_chan(&mut recv).await {
|
while let Ok(ret) = recv_packet_from_chan(&mut recv).await {
|
||||||
let Err(mut ret) =
|
let disable_relay_data = global_ctx.flags_arc().disable_relay_data;
|
||||||
Self::try_handle_foreign_network_packet(ret, my_peer_id, &peers, &foreign_mgr)
|
let Err(mut ret) = Self::try_handle_foreign_network_packet(
|
||||||
|
ret,
|
||||||
|
my_peer_id,
|
||||||
|
&peers,
|
||||||
|
&foreign_mgr,
|
||||||
|
disable_relay_data,
|
||||||
|
)
|
||||||
.await
|
.await
|
||||||
else {
|
else {
|
||||||
continue;
|
continue;
|
||||||
};
|
};
|
||||||
|
|
||||||
let buf_len = ret.buf_len();
|
let buf_len = ret.buf_len();
|
||||||
|
let is_relay_data_packet = Self::is_relay_data_zc_packet(&ret);
|
||||||
let Some(hdr) = ret.mut_peer_manager_header() else {
|
let Some(hdr) = ret.mut_peer_manager_header() else {
|
||||||
tracing::warn!(?ret, "invalid packet, skip");
|
tracing::warn!(?ret, "invalid packet, skip");
|
||||||
continue;
|
continue;
|
||||||
@@ -944,6 +1002,16 @@ impl PeerManager {
|
|||||||
let packet_type = hdr.packet_type;
|
let packet_type = hdr.packet_type;
|
||||||
let is_encrypted = hdr.is_encrypted();
|
let is_encrypted = hdr.is_encrypted();
|
||||||
if to_peer_id != my_peer_id {
|
if to_peer_id != my_peer_id {
|
||||||
|
if disable_relay_data && is_relay_data_packet {
|
||||||
|
tracing::debug!(
|
||||||
|
?from_peer_id,
|
||||||
|
?to_peer_id,
|
||||||
|
packet_type,
|
||||||
|
"drop forwarded relay data while relay data is disabled"
|
||||||
|
);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
if hdr.forward_counter > 7 {
|
if hdr.forward_counter > 7 {
|
||||||
tracing::warn!(?hdr, "forward counter exceed, drop packet");
|
tracing::warn!(?hdr, "forward counter exceed, drop packet");
|
||||||
continue;
|
continue;
|
||||||
@@ -2080,7 +2148,7 @@ mod tests {
|
|||||||
},
|
},
|
||||||
},
|
},
|
||||||
proto::{
|
proto::{
|
||||||
common::{CompressionAlgoPb, NatType, PeerFeatureFlag},
|
common::{CompressionAlgoPb, NatType},
|
||||||
peer_rpc::SecureAuthLevel,
|
peer_rpc::SecureAuthLevel,
|
||||||
},
|
},
|
||||||
tunnel::{
|
tunnel::{
|
||||||
@@ -2224,6 +2292,84 @@ mod tests {
|
|||||||
assert_eq!(signal.version(), initial_version + 2);
|
assert_eq!(signal.version(), initial_version + 2);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn disable_relay_data_classifies_data_plane_packets_only() {
|
||||||
|
for packet_type in [
|
||||||
|
PacketType::Data,
|
||||||
|
PacketType::KcpSrc,
|
||||||
|
PacketType::KcpDst,
|
||||||
|
PacketType::QuicSrc,
|
||||||
|
PacketType::QuicDst,
|
||||||
|
PacketType::DataWithKcpSrcModified,
|
||||||
|
PacketType::DataWithQuicSrcModified,
|
||||||
|
PacketType::ForeignNetworkPacket,
|
||||||
|
] {
|
||||||
|
assert!(PeerManager::is_relay_data_packet(packet_type as u8));
|
||||||
|
}
|
||||||
|
|
||||||
|
for packet_type in [
|
||||||
|
PacketType::RpcReq,
|
||||||
|
PacketType::RpcResp,
|
||||||
|
PacketType::Ping,
|
||||||
|
PacketType::Pong,
|
||||||
|
PacketType::HandShake,
|
||||||
|
PacketType::NoiseHandshakeMsg1,
|
||||||
|
PacketType::NoiseHandshakeMsg2,
|
||||||
|
PacketType::NoiseHandshakeMsg3,
|
||||||
|
PacketType::RelayHandshake,
|
||||||
|
PacketType::RelayHandshakeAck,
|
||||||
|
] {
|
||||||
|
assert!(!PeerManager::is_relay_data_packet(packet_type as u8));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn disable_relay_data_inspects_foreign_network_inner_packet_type() {
|
||||||
|
let network_name = "net1".to_string();
|
||||||
|
|
||||||
|
let mut rpc_packet = ZCPacket::new_with_payload(b"rpc");
|
||||||
|
rpc_packet.fill_peer_manager_hdr(1, 2, PacketType::RpcReq as u8);
|
||||||
|
let mut foreign_rpc_packet =
|
||||||
|
ZCPacket::new_for_foreign_network(&network_name, 2, &rpc_packet);
|
||||||
|
foreign_rpc_packet.fill_peer_manager_hdr(10, 20, PacketType::ForeignNetworkPacket as u8);
|
||||||
|
|
||||||
|
assert_eq!(
|
||||||
|
foreign_rpc_packet.foreign_network_inner_packet_type(),
|
||||||
|
Some(PacketType::RpcReq as u8)
|
||||||
|
);
|
||||||
|
assert!(!PeerManager::is_relay_data_zc_packet(&foreign_rpc_packet));
|
||||||
|
|
||||||
|
let mut data_packet = ZCPacket::new_with_payload(b"data");
|
||||||
|
data_packet.fill_peer_manager_hdr(1, 2, PacketType::Data as u8);
|
||||||
|
let mut foreign_data_packet =
|
||||||
|
ZCPacket::new_for_foreign_network(&network_name, 2, &data_packet);
|
||||||
|
foreign_data_packet.fill_peer_manager_hdr(10, 20, PacketType::ForeignNetworkPacket as u8);
|
||||||
|
|
||||||
|
assert_eq!(
|
||||||
|
foreign_data_packet.foreign_network_inner_packet_type(),
|
||||||
|
Some(PacketType::Data as u8)
|
||||||
|
);
|
||||||
|
assert!(PeerManager::is_relay_data_zc_packet(&foreign_data_packet));
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn non_whitelisted_network_avoid_relay_survives_disable_relay_data_toggle() {
|
||||||
|
let global_ctx = get_mock_global_ctx();
|
||||||
|
let mut flags = global_ctx.get_flags();
|
||||||
|
flags.disable_relay_data = true;
|
||||||
|
flags.relay_network_whitelist = "other-network".to_string();
|
||||||
|
global_ctx.set_flags(flags);
|
||||||
|
|
||||||
|
let (packet_send, _packet_recv) = create_packet_recv_chan();
|
||||||
|
let _peer_mgr = PeerManager::new(RouteAlgoType::Ospf, global_ctx.clone(), packet_send);
|
||||||
|
|
||||||
|
let mut flags = global_ctx.get_flags();
|
||||||
|
flags.disable_relay_data = false;
|
||||||
|
global_ctx.set_flags(flags);
|
||||||
|
|
||||||
|
assert!(global_ctx.get_feature_flags().avoid_relay_data);
|
||||||
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
async fn send_msg_internal_does_not_record_tx_metrics_on_failed_delivery() {
|
async fn send_msg_internal_does_not_record_tx_metrics_on_failed_delivery() {
|
||||||
let peer_mgr = create_mock_peer_manager_with_mock_stun(NatType::Unknown).await;
|
let peer_mgr = create_mock_peer_manager_with_mock_stun(NatType::Unknown).await;
|
||||||
@@ -3121,10 +3267,7 @@ mod tests {
|
|||||||
// when b's avoid_relay_data is true, a->c should route through d and e, cost is 3
|
// when b's avoid_relay_data is true, a->c should route through d and e, cost is 3
|
||||||
peer_mgr_b
|
peer_mgr_b
|
||||||
.get_global_ctx()
|
.get_global_ctx()
|
||||||
.set_feature_flags(PeerFeatureFlag {
|
.set_avoid_relay_data_preference(true);
|
||||||
avoid_relay_data: true,
|
|
||||||
..Default::default()
|
|
||||||
});
|
|
||||||
tokio::time::sleep(Duration::from_secs(2)).await;
|
tokio::time::sleep(Duration::from_secs(2)).await;
|
||||||
if wait_route_appear_with_cost(peer_mgr_a.clone(), peer_mgr_c.my_peer_id, Some(3))
|
if wait_route_appear_with_cost(peer_mgr_a.clone(), peer_mgr_c.my_peer_id, Some(3))
|
||||||
.await
|
.await
|
||||||
|
|||||||
@@ -1228,6 +1228,25 @@ impl SyncedRouteInfo {
|
|||||||
Vec<PeerId>,
|
Vec<PeerId>,
|
||||||
HashMap<Vec<u8>, crate::common::global_ctx::TrustedKeyMetadata>,
|
HashMap<Vec<u8>, crate::common::global_ctx::TrustedKeyMetadata>,
|
||||||
)
|
)
|
||||||
|
where
|
||||||
|
F: FnMut(PeerId) -> bool,
|
||||||
|
{
|
||||||
|
self.verify_and_update_credential_trusts_with_active_peers_protecting(
|
||||||
|
network_secret,
|
||||||
|
is_peer_active,
|
||||||
|
None,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
|
fn verify_and_update_credential_trusts_with_active_peers_protecting<F>(
|
||||||
|
&self,
|
||||||
|
network_secret: Option<&str>,
|
||||||
|
is_peer_active: F,
|
||||||
|
protected_peer_id: Option<PeerId>,
|
||||||
|
) -> (
|
||||||
|
Vec<PeerId>,
|
||||||
|
HashMap<Vec<u8>, crate::common::global_ctx::TrustedKeyMetadata>,
|
||||||
|
)
|
||||||
where
|
where
|
||||||
F: FnMut(PeerId) -> bool,
|
F: FnMut(PeerId) -> bool,
|
||||||
{
|
{
|
||||||
@@ -1248,6 +1267,9 @@ impl SyncedRouteInfo {
|
|||||||
let mut untrusted_peers =
|
let mut untrusted_peers =
|
||||||
Self::collect_revoked_credential_peers(&peer_infos, &prev_trusted, &all_trusted);
|
Self::collect_revoked_credential_peers(&peer_infos, &prev_trusted, &all_trusted);
|
||||||
untrusted_peers.extend(duplicate_untrusted_peers);
|
untrusted_peers.extend(duplicate_untrusted_peers);
|
||||||
|
if let Some(protected_peer_id) = protected_peer_id {
|
||||||
|
untrusted_peers.remove(&protected_peer_id);
|
||||||
|
}
|
||||||
|
|
||||||
// Remove untrusted peers from peer_infos so they won't appear in route graph
|
// Remove untrusted peers from peer_infos so they won't appear in route graph
|
||||||
if !untrusted_peers.is_empty() {
|
if !untrusted_peers.is_empty() {
|
||||||
@@ -2735,7 +2757,11 @@ impl PeerRouteServiceImpl {
|
|||||||
let network_identity = self.global_ctx.get_network_identity();
|
let network_identity = self.global_ctx.get_network_identity();
|
||||||
let (untrusted, global_trusted_keys) = self
|
let (untrusted, global_trusted_keys) = self
|
||||||
.synced_route_info
|
.synced_route_info
|
||||||
.verify_and_update_credential_trusts(network_identity.network_secret.as_deref());
|
.verify_and_update_credential_trusts_with_active_peers_protecting(
|
||||||
|
network_identity.network_secret.as_deref(),
|
||||||
|
|_| true,
|
||||||
|
Some(self.my_peer_id),
|
||||||
|
);
|
||||||
self.global_ctx
|
self.global_ctx
|
||||||
.update_trusted_keys(global_trusted_keys, &network_identity.network_name);
|
.update_trusted_keys(global_trusted_keys, &network_identity.network_name);
|
||||||
|
|
||||||
@@ -2751,9 +2777,10 @@ impl PeerRouteServiceImpl {
|
|||||||
|
|
||||||
let (untrusted, global_trusted_keys) = self
|
let (untrusted, global_trusted_keys) = self
|
||||||
.synced_route_info
|
.synced_route_info
|
||||||
.verify_and_update_credential_trusts_with_active_peers(
|
.verify_and_update_credential_trusts_with_active_peers_protecting(
|
||||||
network_identity.network_secret.as_deref(),
|
network_identity.network_secret.as_deref(),
|
||||||
|peer_id| self.is_active_non_reusable_credential_peer(peer_id),
|
|peer_id| self.is_active_non_reusable_credential_peer(peer_id),
|
||||||
|
Some(self.my_peer_id),
|
||||||
);
|
);
|
||||||
self.global_ctx
|
self.global_ctx
|
||||||
.update_trusted_keys(global_trusted_keys, &network_identity.network_name);
|
.update_trusted_keys(global_trusted_keys, &network_identity.network_name);
|
||||||
@@ -5047,6 +5074,58 @@ mod tests {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn credential_trust_refresh_does_not_remove_self_peer() {
|
||||||
|
let my_peer_id = 11;
|
||||||
|
let remote_peer_id = 12;
|
||||||
|
let credential_key = vec![8; 32];
|
||||||
|
let service_impl = PeerRouteServiceImpl::new(my_peer_id, get_mock_global_ctx());
|
||||||
|
|
||||||
|
let self_info = make_credential_route_peer_info(my_peer_id, &credential_key);
|
||||||
|
let remote_info = make_credential_route_peer_info(remote_peer_id, &credential_key);
|
||||||
|
|
||||||
|
{
|
||||||
|
let mut guard = service_impl.synced_route_info.peer_infos.write();
|
||||||
|
guard.insert(self_info.peer_id, self_info);
|
||||||
|
guard.insert(remote_info.peer_id, remote_info);
|
||||||
|
}
|
||||||
|
service_impl
|
||||||
|
.synced_route_info
|
||||||
|
.trusted_credential_pubkeys
|
||||||
|
.insert(
|
||||||
|
credential_key.clone(),
|
||||||
|
TrustedCredentialPubkey {
|
||||||
|
pubkey: credential_key,
|
||||||
|
expiry_unix: i64::MAX,
|
||||||
|
..Default::default()
|
||||||
|
},
|
||||||
|
);
|
||||||
|
|
||||||
|
let (untrusted_peers, _) = service_impl
|
||||||
|
.synced_route_info
|
||||||
|
.verify_and_update_credential_trusts_with_active_peers_protecting(
|
||||||
|
None,
|
||||||
|
|_| true,
|
||||||
|
Some(my_peer_id),
|
||||||
|
);
|
||||||
|
|
||||||
|
assert_eq!(untrusted_peers, vec![remote_peer_id]);
|
||||||
|
assert!(
|
||||||
|
service_impl
|
||||||
|
.synced_route_info
|
||||||
|
.peer_infos
|
||||||
|
.read()
|
||||||
|
.contains_key(&my_peer_id)
|
||||||
|
);
|
||||||
|
assert!(
|
||||||
|
!service_impl
|
||||||
|
.synced_route_info
|
||||||
|
.peer_infos
|
||||||
|
.read()
|
||||||
|
.contains_key(&remote_peer_id)
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
async fn credential_refresh_rebuilds_reachability_before_owner_election() {
|
async fn credential_refresh_rebuilds_reachability_before_owner_election() {
|
||||||
const NETWORK_SECRET: &str = "sec1";
|
const NETWORK_SECRET: &str = "sec1";
|
||||||
|
|||||||
@@ -7,7 +7,10 @@ use anyhow::anyhow;
|
|||||||
use dashmap::DashMap;
|
use dashmap::DashMap;
|
||||||
|
|
||||||
use super::secure_datagram::{SecureDatagramDirection, SecureDatagramSession};
|
use super::secure_datagram::{SecureDatagramDirection, SecureDatagramSession};
|
||||||
use crate::{common::PeerId, tunnel::packet_def::ZCPacket};
|
use crate::{
|
||||||
|
common::{PeerId, shrink_dashmap},
|
||||||
|
tunnel::packet_def::ZCPacket,
|
||||||
|
};
|
||||||
|
|
||||||
pub struct UpsertResponderSessionReturn {
|
pub struct UpsertResponderSessionReturn {
|
||||||
pub session: Arc<PeerSession>,
|
pub session: Arc<PeerSession>,
|
||||||
@@ -78,6 +81,7 @@ impl PeerSessionStore {
|
|||||||
pub fn evict_unused_sessions(&self) {
|
pub fn evict_unused_sessions(&self) {
|
||||||
self.sessions
|
self.sessions
|
||||||
.retain(|_key, session| Arc::strong_count(session) > 1);
|
.retain(|_key, session| Arc::strong_count(session) > 1);
|
||||||
|
shrink_dashmap(&self.sessions, None);
|
||||||
}
|
}
|
||||||
|
|
||||||
#[tracing::instrument(skip(self))]
|
#[tracing::instrument(skip(self))]
|
||||||
|
|||||||
@@ -9,7 +9,7 @@ use tokio::time::{Duration, timeout};
|
|||||||
use crate::peers::foreign_network_client::ForeignNetworkClient;
|
use crate::peers::foreign_network_client::ForeignNetworkClient;
|
||||||
use crate::{
|
use crate::{
|
||||||
common::error::Error,
|
common::error::Error,
|
||||||
common::{PeerId, global_ctx::ArcGlobalCtx},
|
common::{PeerId, global_ctx::ArcGlobalCtx, shrink_dashmap},
|
||||||
peers::peer_map::PeerMap,
|
peers::peer_map::PeerMap,
|
||||||
peers::peer_session::{PeerSession, PeerSessionAction, PeerSessionStore, SessionKey},
|
peers::peer_session::{PeerSession, PeerSessionAction, PeerSessionStore, SessionKey},
|
||||||
peers::route_trait::NextHopPolicy,
|
peers::route_trait::NextHopPolicy,
|
||||||
@@ -652,6 +652,10 @@ impl RelayPeerMap {
|
|||||||
self.handshake_locks.remove(&peer_id);
|
self.handshake_locks.remove(&peer_id);
|
||||||
self.pending_packets.remove(&peer_id);
|
self.pending_packets.remove(&peer_id);
|
||||||
}
|
}
|
||||||
|
shrink_dashmap(&self.states, None);
|
||||||
|
shrink_dashmap(&self.pending_handshakes, None);
|
||||||
|
shrink_dashmap(&self.handshake_locks, None);
|
||||||
|
shrink_dashmap(&self.pending_packets, None);
|
||||||
}
|
}
|
||||||
|
|
||||||
pub fn has_state(&self, peer_id: PeerId) -> bool {
|
pub fn has_state(&self, peer_id: PeerId) -> bool {
|
||||||
@@ -679,6 +683,10 @@ impl RelayPeerMap {
|
|||||||
self.pending_handshakes.remove(&peer_id);
|
self.pending_handshakes.remove(&peer_id);
|
||||||
self.handshake_locks.remove(&peer_id);
|
self.handshake_locks.remove(&peer_id);
|
||||||
self.pending_packets.remove(&peer_id);
|
self.pending_packets.remove(&peer_id);
|
||||||
|
shrink_dashmap(&self.states, None);
|
||||||
|
shrink_dashmap(&self.pending_handshakes, None);
|
||||||
|
shrink_dashmap(&self.handshake_locks, None);
|
||||||
|
shrink_dashmap(&self.pending_packets, None);
|
||||||
|
|
||||||
tracing::debug!(?peer_id, "RelayPeerMap removed peer relay state");
|
tracing::debug!(?peer_id, "RelayPeerMap removed peer relay state");
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -201,6 +201,11 @@ impl LogicalTrafficMetrics {
|
|||||||
self.per_peer.len()
|
self.per_peer.len()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[cfg(test)]
|
||||||
|
fn contains_peer_cache(&self, peer_id: PeerId) -> bool {
|
||||||
|
self.per_peer.contains_key(&peer_id)
|
||||||
|
}
|
||||||
|
|
||||||
fn build_peer_counters(&self, instance_id: &str) -> TrafficCounters {
|
fn build_peer_counters(&self, instance_id: &str) -> TrafficCounters {
|
||||||
let instance_label = match self.label_kind {
|
let instance_label = match self.label_kind {
|
||||||
InstanceLabelKind::To => LabelType::ToInstanceId(instance_id.to_string()),
|
InstanceLabelKind::To => LabelType::ToInstanceId(instance_id.to_string()),
|
||||||
@@ -241,6 +246,13 @@ pub(crate) fn traffic_kind(packet_type: u8) -> TrafficKind {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
pub(crate) fn is_relay_data_packet_type(packet_type: u8) -> bool {
|
||||||
|
// Relay handshakes are control-plane setup; payload data is blocked by its
|
||||||
|
// original packet type after the session exists.
|
||||||
|
traffic_kind(packet_type) == TrafficKind::Data
|
||||||
|
|| packet_type == PacketType::ForeignNetworkPacket as u8
|
||||||
|
}
|
||||||
|
|
||||||
#[derive(Clone)]
|
#[derive(Clone)]
|
||||||
struct TrafficMetricGroup {
|
struct TrafficMetricGroup {
|
||||||
data: Arc<LogicalTrafficMetrics>,
|
data: Arc<LogicalTrafficMetrics>,
|
||||||
@@ -326,6 +338,14 @@ impl TrafficMetricRecorder {
|
|||||||
self.rx_metrics.control.clear_peer_cache();
|
self.rx_metrics.control.clear_peer_cache();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[cfg(test)]
|
||||||
|
pub(crate) fn contains_peer_cache(&self, peer_id: PeerId) -> bool {
|
||||||
|
self.tx_metrics.data.contains_peer_cache(peer_id)
|
||||||
|
|| self.tx_metrics.control.contains_peer_cache(peer_id)
|
||||||
|
|| self.rx_metrics.data.contains_peer_cache(peer_id)
|
||||||
|
|| self.rx_metrics.control.contains_peer_cache(peer_id)
|
||||||
|
}
|
||||||
|
|
||||||
fn resolve_instance_id(&self, peer_id: PeerId) -> BoxFuture<'static, Option<String>> {
|
fn resolve_instance_id(&self, peer_id: PeerId) -> BoxFuture<'static, Option<String>> {
|
||||||
(self.resolve_instance_id)(peer_id)
|
(self.resolve_instance_id)(peer_id)
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -27,6 +27,7 @@ message InstanceConfigPatch {
|
|||||||
optional bool ipv6_public_addr_provider = 11;
|
optional bool ipv6_public_addr_provider = 11;
|
||||||
optional bool ipv6_public_addr_auto = 12;
|
optional bool ipv6_public_addr_auto = 12;
|
||||||
optional string ipv6_public_addr_prefix = 13;
|
optional string ipv6_public_addr_prefix = 13;
|
||||||
|
optional bool disable_relay_data = 14;
|
||||||
}
|
}
|
||||||
|
|
||||||
message PortForwardPatch {
|
message PortForwardPatch {
|
||||||
|
|||||||
@@ -99,6 +99,7 @@ message NetworkConfig {
|
|||||||
optional bool ipv6_public_addr_provider = 62;
|
optional bool ipv6_public_addr_provider = 62;
|
||||||
optional bool ipv6_public_addr_auto = 63;
|
optional bool ipv6_public_addr_auto = 63;
|
||||||
optional string ipv6_public_addr_prefix = 64;
|
optional string ipv6_public_addr_prefix = 64;
|
||||||
|
optional bool disable_relay_data = 65;
|
||||||
}
|
}
|
||||||
|
|
||||||
message PortForwardConfig {
|
message PortForwardConfig {
|
||||||
|
|||||||
@@ -75,6 +75,7 @@ message FlagsInConfig {
|
|||||||
bool need_p2p = 38;
|
bool need_p2p = 38;
|
||||||
uint64 instance_recv_bps_limit = 39;
|
uint64 instance_recv_bps_limit = 39;
|
||||||
bool disable_upnp = 40;
|
bool disable_upnp = 40;
|
||||||
|
bool disable_relay_data = 41;
|
||||||
}
|
}
|
||||||
|
|
||||||
message RpcDescriptor {
|
message RpcDescriptor {
|
||||||
|
|||||||
@@ -14,13 +14,17 @@ use crate::{
|
|||||||
},
|
},
|
||||||
instance::instance::Instance,
|
instance::instance::Instance,
|
||||||
tests::three_node::{generate_secure_mode_config, generate_secure_mode_config_with_key},
|
tests::three_node::{generate_secure_mode_config, generate_secure_mode_config_with_key},
|
||||||
tunnel::{common::tests::wait_for_condition, tcp::TcpTunnelConnector},
|
tunnel::{common::tests::wait_for_condition, tcp::TcpTunnelConnector, udp::UdpTunnelConnector},
|
||||||
};
|
};
|
||||||
|
|
||||||
use super::{add_ns_to_bridge, create_netns, del_netns, drop_insts, ping_test};
|
use super::{add_ns_to_bridge, create_netns, del_netns, drop_insts, ping_test};
|
||||||
|
|
||||||
use rstest::rstest;
|
use rstest::rstest;
|
||||||
|
|
||||||
|
const PUBLIC_SERVER_NETWORK_NAME: &str = "__public_server__";
|
||||||
|
const PUBLIC_SERVER_SHARED_SECRET: &str = "public-server-shared-secret";
|
||||||
|
const NEED_P2P_ADMIN_NETWORK_NAME: &str = "need_p2p_credential_test_network";
|
||||||
|
|
||||||
/// Prepare network namespaces for credential tests
|
/// Prepare network namespaces for credential tests
|
||||||
/// Topology:
|
/// Topology:
|
||||||
/// br_a (10.1.1.0/24): ns_adm (10.1.1.1), ns_c1 (10.1.1.2), ns_c2 (10.1.1.3), ns_c3 (10.1.1.4), ns_c4 (10.1.1.5)
|
/// br_a (10.1.1.0/24): ns_adm (10.1.1.1), ns_c1 (10.1.1.2), ns_c2 (10.1.1.3), ns_c3 (10.1.1.4), ns_c4 (10.1.1.5)
|
||||||
@@ -221,6 +225,328 @@ fn create_shared_config(
|
|||||||
config
|
config
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn create_public_server_config() -> TomlConfigLoader {
|
||||||
|
let config = TomlConfigLoader::default();
|
||||||
|
config.set_inst_name(PUBLIC_SERVER_NETWORK_NAME.to_string());
|
||||||
|
config.set_hostname(Some("public-server".to_string()));
|
||||||
|
config.set_netns(Some("ns_adm".to_string()));
|
||||||
|
config.set_listeners(vec!["udp://0.0.0.0:11010".parse().unwrap()]);
|
||||||
|
config.set_network_identity(NetworkIdentity::new(
|
||||||
|
PUBLIC_SERVER_NETWORK_NAME.to_string(),
|
||||||
|
PUBLIC_SERVER_SHARED_SECRET.to_string(),
|
||||||
|
));
|
||||||
|
config.set_secure_mode(Some(generate_secure_mode_config()));
|
||||||
|
|
||||||
|
let mut flags = config.get_flags();
|
||||||
|
flags.no_tun = true;
|
||||||
|
flags.private_mode = true;
|
||||||
|
flags.relay_all_peer_rpc = true;
|
||||||
|
flags.relay_network_whitelist = "".to_string();
|
||||||
|
config.set_flags(flags);
|
||||||
|
|
||||||
|
config
|
||||||
|
}
|
||||||
|
|
||||||
|
fn create_need_p2p_admin_config(listener_scheme: &str) -> TomlConfigLoader {
|
||||||
|
let config = TomlConfigLoader::default();
|
||||||
|
config.set_inst_name(NEED_P2P_ADMIN_NETWORK_NAME.to_string());
|
||||||
|
config.set_hostname(Some("need-p2p-admin".to_string()));
|
||||||
|
config.set_netns(Some("ns_c3".to_string()));
|
||||||
|
config.set_listeners(vec![
|
||||||
|
format!("{listener_scheme}://0.0.0.0:0").parse().unwrap(),
|
||||||
|
]);
|
||||||
|
config.set_network_identity(NetworkIdentity::new(
|
||||||
|
NEED_P2P_ADMIN_NETWORK_NAME.to_string(),
|
||||||
|
PUBLIC_SERVER_SHARED_SECRET.to_string(),
|
||||||
|
));
|
||||||
|
config.set_secure_mode(Some(generate_secure_mode_config()));
|
||||||
|
|
||||||
|
let mut flags = config.get_flags();
|
||||||
|
flags.no_tun = true;
|
||||||
|
flags.relay_all_peer_rpc = true;
|
||||||
|
flags.need_p2p = true;
|
||||||
|
flags.disable_udp_hole_punching = true;
|
||||||
|
flags.disable_tcp_hole_punching = true;
|
||||||
|
flags.disable_sym_hole_punching = true;
|
||||||
|
config.set_flags(flags);
|
||||||
|
|
||||||
|
config
|
||||||
|
}
|
||||||
|
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
|
fn create_public_server_credential_config(
|
||||||
|
credential_secret: &str,
|
||||||
|
inst_name: &str,
|
||||||
|
hostname: &str,
|
||||||
|
ns: &str,
|
||||||
|
ipv4: &str,
|
||||||
|
ipv6: &str,
|
||||||
|
tcp_listener_port: u16,
|
||||||
|
udp_listener_port: u16,
|
||||||
|
proxy_cidrs: &[&str],
|
||||||
|
) -> TomlConfigLoader {
|
||||||
|
let config = create_credential_config_from_secret(
|
||||||
|
NEED_P2P_ADMIN_NETWORK_NAME.to_string(),
|
||||||
|
credential_secret,
|
||||||
|
inst_name,
|
||||||
|
Some(ns),
|
||||||
|
ipv4,
|
||||||
|
ipv6,
|
||||||
|
);
|
||||||
|
config.set_hostname(Some(hostname.to_string()));
|
||||||
|
config.set_listeners(vec![
|
||||||
|
format!("tcp://0.0.0.0:{tcp_listener_port}")
|
||||||
|
.parse()
|
||||||
|
.unwrap(),
|
||||||
|
format!("udp://0.0.0.0:{udp_listener_port}")
|
||||||
|
.parse()
|
||||||
|
.unwrap(),
|
||||||
|
]);
|
||||||
|
for cidr in proxy_cidrs {
|
||||||
|
config
|
||||||
|
.add_proxy_cidr((*cidr).parse().unwrap(), None)
|
||||||
|
.unwrap();
|
||||||
|
}
|
||||||
|
|
||||||
|
let mut flags = config.get_flags();
|
||||||
|
flags.disable_p2p = true;
|
||||||
|
config.set_flags(flags);
|
||||||
|
|
||||||
|
config
|
||||||
|
}
|
||||||
|
|
||||||
|
async fn wait_direct_peer(inst: &Instance, peer_id: u32, timeout: Duration, label: &str) {
|
||||||
|
wait_for_condition(
|
||||||
|
|| async {
|
||||||
|
let peers = inst.get_peer_manager().get_peer_map().list_peers();
|
||||||
|
let connected = peers.contains(&peer_id);
|
||||||
|
println!("{label}: direct peers={:?}, target={}", peers, peer_id);
|
||||||
|
connected
|
||||||
|
},
|
||||||
|
timeout,
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
|
||||||
|
async fn wait_running_listener(inst: &Instance, scheme: &str, timeout: Duration, label: &str) {
|
||||||
|
wait_for_condition(
|
||||||
|
|| async {
|
||||||
|
let listeners = inst.get_global_ctx().get_running_listeners();
|
||||||
|
let matched = listeners.iter().any(|listener| {
|
||||||
|
listener.scheme() == scheme && listener.port().is_some_and(|p| p != 0)
|
||||||
|
});
|
||||||
|
println!("{label}: running listeners={:?}", listeners);
|
||||||
|
matched
|
||||||
|
},
|
||||||
|
timeout,
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
|
||||||
|
async fn wait_route_cost(inst: &Instance, peer_id: u32, cost: i32, timeout: Duration, label: &str) {
|
||||||
|
wait_for_condition(
|
||||||
|
|| async {
|
||||||
|
let routes = inst.get_peer_manager().list_routes().await;
|
||||||
|
let matched = routes
|
||||||
|
.iter()
|
||||||
|
.any(|route| route.peer_id == peer_id && route.cost == cost);
|
||||||
|
println!(
|
||||||
|
"{label}: routes={:?}, target={}, cost={}",
|
||||||
|
routes
|
||||||
|
.iter()
|
||||||
|
.map(|route| (route.peer_id, route.cost))
|
||||||
|
.collect::<Vec<_>>(),
|
||||||
|
peer_id,
|
||||||
|
cost
|
||||||
|
);
|
||||||
|
matched
|
||||||
|
},
|
||||||
|
timeout,
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
|
||||||
|
async fn wait_foreign_network_count(inst: &Instance, expected: usize, timeout: Duration) {
|
||||||
|
wait_for_condition(
|
||||||
|
|| async {
|
||||||
|
let foreign_networks = inst
|
||||||
|
.get_peer_manager()
|
||||||
|
.get_foreign_network_manager()
|
||||||
|
.list_foreign_networks()
|
||||||
|
.await
|
||||||
|
.foreign_networks;
|
||||||
|
println!("foreign networks: {:?}", foreign_networks);
|
||||||
|
foreign_networks.len() == expected
|
||||||
|
},
|
||||||
|
timeout,
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Regression coverage for a public-server-mediated credential topology:
|
||||||
|
/// Public server <- admin peer (need_p2p) <- two credential peers.
|
||||||
|
///
|
||||||
|
/// Credential peers set `disable_p2p=true`, while the admin peer advertises `need_p2p=true`.
|
||||||
|
/// The credential peers should still proactively build direct peers with the admin peer through
|
||||||
|
/// peer RPC forwarded by the public server, even when the admin listener binds an ephemeral port.
|
||||||
|
#[rstest]
|
||||||
|
#[case("quic")]
|
||||||
|
#[case("wss")]
|
||||||
|
#[case("tcp")]
|
||||||
|
#[case("udp")]
|
||||||
|
#[tokio::test]
|
||||||
|
#[serial_test::serial]
|
||||||
|
async fn credential_peers_p2p_to_need_p2p_admin_through_public_server(
|
||||||
|
#[case] admin_listener_scheme: &str,
|
||||||
|
) {
|
||||||
|
prepare_credential_network();
|
||||||
|
|
||||||
|
let mut public_server_inst = Instance::new(create_public_server_config());
|
||||||
|
public_server_inst.run().await.unwrap();
|
||||||
|
|
||||||
|
let mut admin_inst = Instance::new(create_need_p2p_admin_config(admin_listener_scheme));
|
||||||
|
admin_inst.run().await.unwrap();
|
||||||
|
wait_running_listener(
|
||||||
|
&admin_inst,
|
||||||
|
admin_listener_scheme,
|
||||||
|
Duration::from_secs(10),
|
||||||
|
"admin ephemeral listener",
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
admin_inst
|
||||||
|
.get_conn_manager()
|
||||||
|
.add_connector(UdpTunnelConnector::new(
|
||||||
|
"udp://10.1.1.1:11010".parse().unwrap(),
|
||||||
|
));
|
||||||
|
|
||||||
|
wait_foreign_network_count(&public_server_inst, 1, Duration::from_secs(10)).await;
|
||||||
|
|
||||||
|
let (_credential_a_id, credential_a_secret) = admin_inst
|
||||||
|
.get_global_ctx()
|
||||||
|
.get_credential_manager()
|
||||||
|
.generate_credential_with_options(
|
||||||
|
vec![],
|
||||||
|
false,
|
||||||
|
vec!["10.1.0.0/24".to_string()],
|
||||||
|
Duration::from_secs(3600),
|
||||||
|
Some("credential-peer-a".to_string()),
|
||||||
|
false,
|
||||||
|
);
|
||||||
|
let (_credential_b_id, credential_b_secret) = admin_inst
|
||||||
|
.get_global_ctx()
|
||||||
|
.get_credential_manager()
|
||||||
|
.generate_credential_with_options(
|
||||||
|
vec![],
|
||||||
|
false,
|
||||||
|
vec![],
|
||||||
|
Duration::from_secs(3600),
|
||||||
|
Some("credential-peer-b".to_string()),
|
||||||
|
false,
|
||||||
|
);
|
||||||
|
admin_inst
|
||||||
|
.get_global_ctx()
|
||||||
|
.issue_event(GlobalCtxEvent::CredentialChanged);
|
||||||
|
|
||||||
|
wait_foreign_network_count(&public_server_inst, 1, Duration::from_secs(10)).await;
|
||||||
|
|
||||||
|
let mut credential_a_inst = Instance::new(create_public_server_credential_config(
|
||||||
|
&credential_a_secret,
|
||||||
|
"credential-peer-a",
|
||||||
|
"credential-a",
|
||||||
|
"ns_c1",
|
||||||
|
"10.154.0.1",
|
||||||
|
"fd00::1/64",
|
||||||
|
11030,
|
||||||
|
11031,
|
||||||
|
&["10.1.0.0/24"],
|
||||||
|
));
|
||||||
|
let mut credential_b_inst = Instance::new(create_public_server_credential_config(
|
||||||
|
&credential_b_secret,
|
||||||
|
"credential-peer-b",
|
||||||
|
"credential-b",
|
||||||
|
"ns_c2",
|
||||||
|
"10.154.0.2",
|
||||||
|
"fd00::2/64",
|
||||||
|
11040,
|
||||||
|
11041,
|
||||||
|
&[],
|
||||||
|
));
|
||||||
|
credential_a_inst.run().await.unwrap();
|
||||||
|
credential_b_inst.run().await.unwrap();
|
||||||
|
|
||||||
|
credential_a_inst
|
||||||
|
.get_conn_manager()
|
||||||
|
.add_connector(UdpTunnelConnector::new(
|
||||||
|
"udp://10.1.1.1:11010".parse().unwrap(),
|
||||||
|
));
|
||||||
|
credential_b_inst
|
||||||
|
.get_conn_manager()
|
||||||
|
.add_connector(UdpTunnelConnector::new(
|
||||||
|
"udp://10.1.1.1:11010".parse().unwrap(),
|
||||||
|
));
|
||||||
|
|
||||||
|
let admin_peer_id = admin_inst.peer_id();
|
||||||
|
let credential_a_peer_id = credential_a_inst.peer_id();
|
||||||
|
let credential_b_peer_id = credential_b_inst.peer_id();
|
||||||
|
println!(
|
||||||
|
"admin={}, credential_a={}, credential_b={}, admin_listener_scheme={}",
|
||||||
|
admin_peer_id, credential_a_peer_id, credential_b_peer_id, admin_listener_scheme
|
||||||
|
);
|
||||||
|
|
||||||
|
wait_direct_peer(
|
||||||
|
&credential_a_inst,
|
||||||
|
admin_peer_id,
|
||||||
|
Duration::from_secs(30),
|
||||||
|
"credential_a -> admin",
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
wait_direct_peer(
|
||||||
|
&credential_b_inst,
|
||||||
|
admin_peer_id,
|
||||||
|
Duration::from_secs(30),
|
||||||
|
"credential_b -> admin",
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
wait_direct_peer(
|
||||||
|
&admin_inst,
|
||||||
|
credential_a_peer_id,
|
||||||
|
Duration::from_secs(10),
|
||||||
|
"admin -> credential_a",
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
wait_direct_peer(
|
||||||
|
&admin_inst,
|
||||||
|
credential_b_peer_id,
|
||||||
|
Duration::from_secs(10),
|
||||||
|
"admin -> credential_b",
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
wait_route_cost(
|
||||||
|
&credential_a_inst,
|
||||||
|
admin_peer_id,
|
||||||
|
1,
|
||||||
|
Duration::from_secs(10),
|
||||||
|
"credential_a route to admin",
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
wait_route_cost(
|
||||||
|
&credential_b_inst,
|
||||||
|
admin_peer_id,
|
||||||
|
1,
|
||||||
|
Duration::from_secs(10),
|
||||||
|
"credential_b route to admin",
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
|
||||||
|
drop_insts(vec![
|
||||||
|
public_server_inst,
|
||||||
|
admin_inst,
|
||||||
|
credential_a_inst,
|
||||||
|
credential_b_inst,
|
||||||
|
])
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
|
||||||
fn create_generated_credential_config(
|
fn create_generated_credential_config(
|
||||||
admin_inst: &Instance,
|
admin_inst: &Instance,
|
||||||
inst_name: &str,
|
inst_name: &str,
|
||||||
@@ -501,10 +827,10 @@ async fn credential_relay_capability(#[case] allow_relay: bool) {
|
|||||||
// Create admin node
|
// Create admin node
|
||||||
let admin_config = create_admin_config("admin", Some("ns_adm"), "10.144.144.1", "fd00::1/64");
|
let admin_config = create_admin_config("admin", Some("ns_adm"), "10.144.144.1", "fd00::1/64");
|
||||||
let mut admin_inst = Instance::new(admin_config);
|
let mut admin_inst = Instance::new(admin_config);
|
||||||
let mut ff = admin_inst.get_global_ctx().get_feature_flags();
|
|
||||||
// if cred c allow relay, we set admin inst avoid relay (if other same-cost path available, admin will not relay data)
|
// if cred c allow relay, we set admin inst avoid relay (if other same-cost path available, admin will not relay data)
|
||||||
ff.avoid_relay_data = allow_relay;
|
admin_inst
|
||||||
admin_inst.get_global_ctx().set_feature_flags(ff);
|
.get_global_ctx()
|
||||||
|
.set_avoid_relay_data_preference(allow_relay);
|
||||||
admin_inst.run().await.unwrap();
|
admin_inst.run().await.unwrap();
|
||||||
|
|
||||||
let admin_peer_id = admin_inst.peer_id();
|
let admin_peer_id = admin_inst.peer_id();
|
||||||
|
|||||||
@@ -3730,6 +3730,153 @@ pub async fn config_patch_test() {
|
|||||||
drop_insts(insts).await;
|
drop_insts(insts).await;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[rstest::rstest]
|
||||||
|
#[tokio::test]
|
||||||
|
#[serial_test::serial]
|
||||||
|
pub async fn config_patch_disable_relay_data_test() {
|
||||||
|
use crate::proto::api::config::InstanceConfigPatch;
|
||||||
|
|
||||||
|
let insts = init_three_node_ex(
|
||||||
|
"udp",
|
||||||
|
|cfg| {
|
||||||
|
cfg.set_ipv6(None);
|
||||||
|
cfg
|
||||||
|
},
|
||||||
|
false,
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
|
||||||
|
let relay_peer_id = insts[1].peer_id();
|
||||||
|
let dst_peer_id = insts[2].peer_id();
|
||||||
|
assert!(!insts[1].get_global_ctx().get_flags().disable_relay_data);
|
||||||
|
assert!(
|
||||||
|
!insts[1]
|
||||||
|
.get_global_ctx()
|
||||||
|
.get_feature_flags()
|
||||||
|
.avoid_relay_data
|
||||||
|
);
|
||||||
|
|
||||||
|
check_route_ex(
|
||||||
|
insts[0].get_peer_manager().list_routes().await,
|
||||||
|
dst_peer_id,
|
||||||
|
|route| {
|
||||||
|
assert_eq!(route.next_hop_peer_id, relay_peer_id);
|
||||||
|
true
|
||||||
|
},
|
||||||
|
);
|
||||||
|
|
||||||
|
wait_for_condition(
|
||||||
|
|| async { ping_test("net_a", "10.144.144.3", None).await },
|
||||||
|
Duration::from_secs(5),
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
|
||||||
|
insts[1]
|
||||||
|
.get_config_patcher()
|
||||||
|
.apply_patch(InstanceConfigPatch {
|
||||||
|
disable_relay_data: Some(true),
|
||||||
|
..Default::default()
|
||||||
|
})
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
|
||||||
|
assert!(insts[1].get_global_ctx().get_flags().disable_relay_data);
|
||||||
|
assert!(
|
||||||
|
insts[1]
|
||||||
|
.get_global_ctx()
|
||||||
|
.config
|
||||||
|
.get_flags()
|
||||||
|
.disable_relay_data
|
||||||
|
);
|
||||||
|
assert!(
|
||||||
|
insts[1]
|
||||||
|
.get_global_ctx()
|
||||||
|
.get_feature_flags()
|
||||||
|
.avoid_relay_data
|
||||||
|
);
|
||||||
|
|
||||||
|
wait_for_condition(
|
||||||
|
|| {
|
||||||
|
let peer_mgr = insts[0].get_peer_manager().clone();
|
||||||
|
async move {
|
||||||
|
peer_mgr.list_routes().await.iter().any(|route| {
|
||||||
|
route.peer_id == relay_peer_id
|
||||||
|
&& route
|
||||||
|
.feature_flag
|
||||||
|
.as_ref()
|
||||||
|
.map(|flag| flag.avoid_relay_data)
|
||||||
|
.unwrap_or(false)
|
||||||
|
})
|
||||||
|
}
|
||||||
|
},
|
||||||
|
Duration::from_secs(5),
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
|
||||||
|
check_route_ex(
|
||||||
|
insts[0].get_peer_manager().list_routes().await,
|
||||||
|
dst_peer_id,
|
||||||
|
|route| {
|
||||||
|
assert_eq!(route.next_hop_peer_id, relay_peer_id);
|
||||||
|
true
|
||||||
|
},
|
||||||
|
);
|
||||||
|
assert!(
|
||||||
|
!ping_test("net_a", "10.144.144.3", None).await,
|
||||||
|
"traffic from inst1 to inst3 should be blocked while inst2 relay data is disabled"
|
||||||
|
);
|
||||||
|
|
||||||
|
insts[1]
|
||||||
|
.get_config_patcher()
|
||||||
|
.apply_patch(InstanceConfigPatch {
|
||||||
|
disable_relay_data: Some(false),
|
||||||
|
..Default::default()
|
||||||
|
})
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
|
||||||
|
assert!(!insts[1].get_global_ctx().get_flags().disable_relay_data);
|
||||||
|
assert!(
|
||||||
|
!insts[1]
|
||||||
|
.get_global_ctx()
|
||||||
|
.config
|
||||||
|
.get_flags()
|
||||||
|
.disable_relay_data
|
||||||
|
);
|
||||||
|
assert!(
|
||||||
|
!insts[1]
|
||||||
|
.get_global_ctx()
|
||||||
|
.get_feature_flags()
|
||||||
|
.avoid_relay_data
|
||||||
|
);
|
||||||
|
|
||||||
|
wait_for_condition(
|
||||||
|
|| {
|
||||||
|
let peer_mgr = insts[0].get_peer_manager().clone();
|
||||||
|
async move {
|
||||||
|
peer_mgr.list_routes().await.iter().any(|route| {
|
||||||
|
route.peer_id == relay_peer_id
|
||||||
|
&& route
|
||||||
|
.feature_flag
|
||||||
|
.as_ref()
|
||||||
|
.map(|flag| !flag.avoid_relay_data)
|
||||||
|
.unwrap_or(false)
|
||||||
|
})
|
||||||
|
}
|
||||||
|
},
|
||||||
|
Duration::from_secs(5),
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
|
||||||
|
wait_for_condition(
|
||||||
|
|| async { ping_test("net_a", "10.144.144.3", None).await },
|
||||||
|
Duration::from_secs(5),
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
|
||||||
|
drop_insts(insts).await;
|
||||||
|
}
|
||||||
|
|
||||||
/// Generate SecureModeConfig with specified x25519 private key
|
/// Generate SecureModeConfig with specified x25519 private key
|
||||||
pub fn generate_secure_mode_config_with_key(
|
pub fn generate_secure_mode_config_with_key(
|
||||||
private_key: &x25519_dalek::StaticSecret,
|
private_key: &x25519_dalek::StaticSecret,
|
||||||
|
|||||||
@@ -57,21 +57,21 @@ cfg_select! {
|
|||||||
pub mod windivert;
|
pub mod windivert;
|
||||||
|
|
||||||
pub fn create_tun(
|
pub fn create_tun(
|
||||||
_interface_name: &str,
|
interface_name: &str,
|
||||||
_src_addr: Option<SocketAddr>,
|
src_addr: Option<SocketAddr>,
|
||||||
local_addr: SocketAddr,
|
dst_addr: SocketAddr,
|
||||||
) -> io::Result<Arc<dyn super::stack::Tun>> {
|
) -> io::Result<Arc<dyn super::stack::Tun>> {
|
||||||
match windivert::WinDivertTun::new(local_addr) {
|
match windivert::WinDivertTun::new(src_addr, dst_addr) {
|
||||||
Ok(tun) => Ok(Arc::new(tun)),
|
Ok(tun) => Ok(Arc::new(tun)),
|
||||||
Err(e) => {
|
Err(e) => {
|
||||||
tracing::warn!(
|
tracing::warn!(
|
||||||
?e,
|
?e,
|
||||||
?local_addr,
|
?dst_addr,
|
||||||
"WinDivertTun init failed, falling back to PnetTun"
|
"WinDivertTun init failed, falling back to PnetTun"
|
||||||
);
|
);
|
||||||
Ok(Arc::new(pnet::PnetTun::new(
|
Ok(Arc::new(pnet::PnetTun::new(
|
||||||
local_addr.to_string().as_str(),
|
interface_name,
|
||||||
pnet::create_packet_filter(None, local_addr),
|
pnet::create_packet_filter(src_addr, dst_addr),
|
||||||
)?))
|
)?))
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -80,15 +80,11 @@ impl Drop for WinDivertTun {
|
|||||||
}
|
}
|
||||||
|
|
||||||
impl WinDivertTun {
|
impl WinDivertTun {
|
||||||
pub fn new(local_addr: SocketAddr) -> io::Result<Self> {
|
pub fn new(src_addr: Option<SocketAddr>, dst_addr: SocketAddr) -> io::Result<Self> {
|
||||||
let (tx, rx) = tokio::sync::mpsc::channel(1024);
|
let (tx, rx) = tokio::sync::mpsc::channel(1024);
|
||||||
|
|
||||||
let ip_filter = match local_addr {
|
let filter = build_filter(src_addr, dst_addr)?;
|
||||||
SocketAddr::V4(addr) => format!("ip.DstAddr == {}", addr.ip()),
|
tracing::debug!(%filter, "WinDivertTun created with filter");
|
||||||
SocketAddr::V6(addr) => format!("ipv6.DstAddr == {}", addr.ip()),
|
|
||||||
};
|
|
||||||
// Filter: DstIP == LocalIP AND TCP.
|
|
||||||
let filter = format!("{} and tcp", ip_filter);
|
|
||||||
|
|
||||||
// Sniff mode: 1 (WINDIVERT_FLAG_SNIFF)
|
// Sniff mode: 1 (WINDIVERT_FLAG_SNIFF)
|
||||||
// Layer: Network (0)
|
// Layer: Network (0)
|
||||||
@@ -143,6 +139,46 @@ impl WinDivertTun {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn build_filter(src_addr: Option<SocketAddr>, dst_addr: SocketAddr) -> io::Result<String> {
|
||||||
|
if let Some(src_addr) = src_addr
|
||||||
|
&& src_addr.is_ipv4() != dst_addr.is_ipv4()
|
||||||
|
{
|
||||||
|
return Err(io::Error::new(
|
||||||
|
io::ErrorKind::InvalidInput,
|
||||||
|
"src/dst addr family mismatch",
|
||||||
|
));
|
||||||
|
}
|
||||||
|
|
||||||
|
let mut filters = Vec::with_capacity(5);
|
||||||
|
filters.push("tcp".to_owned());
|
||||||
|
|
||||||
|
match dst_addr {
|
||||||
|
SocketAddr::V4(addr) => {
|
||||||
|
filters.push(format!("ip.DstAddr == {}", addr.ip()));
|
||||||
|
filters.push(format!("tcp.DstPort == {}", addr.port()));
|
||||||
|
}
|
||||||
|
SocketAddr::V6(addr) => {
|
||||||
|
filters.push(format!("ipv6.DstAddr == {}", addr.ip()));
|
||||||
|
filters.push(format!("tcp.DstPort == {}", addr.port()));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if let Some(src_addr) = src_addr {
|
||||||
|
match src_addr {
|
||||||
|
SocketAddr::V4(addr) => {
|
||||||
|
filters.push(format!("ip.SrcAddr == {}", addr.ip()));
|
||||||
|
filters.push(format!("tcp.SrcPort == {}", addr.port()));
|
||||||
|
}
|
||||||
|
SocketAddr::V6(addr) => {
|
||||||
|
filters.push(format!("ipv6.SrcAddr == {}", addr.ip()));
|
||||||
|
filters.push(format!("tcp.SrcPort == {}", addr.port()));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
Ok(filters.join(" and "))
|
||||||
|
}
|
||||||
|
|
||||||
#[async_trait::async_trait]
|
#[async_trait::async_trait]
|
||||||
impl stack::Tun for WinDivertTun {
|
impl stack::Tun for WinDivertTun {
|
||||||
async fn recv(&self, packet: &mut BytesMut) -> Result<usize, std::io::Error> {
|
async fn recv(&self, packet: &mut BytesMut) -> Result<usize, std::io::Error> {
|
||||||
|
|||||||
@@ -128,7 +128,6 @@ pub fn build_tcp_packet(
|
|||||||
eth_buf.freeze()
|
eth_buf.freeze()
|
||||||
}
|
}
|
||||||
|
|
||||||
#[tracing::instrument(ret)]
|
|
||||||
pub fn parse_ip_packet(
|
pub fn parse_ip_packet(
|
||||||
buf: &Bytes,
|
buf: &Bytes,
|
||||||
) -> Option<(MacAddr, MacAddr, IPPacket<'_>, tcp::TcpPacket<'_>)> {
|
) -> Option<(MacAddr, MacAddr, IPPacket<'_>, tcp::TcpPacket<'_>)> {
|
||||||
|
|||||||
@@ -517,9 +517,12 @@ impl Stack {
|
|||||||
{
|
{
|
||||||
trace!(?tcp_packet, "Received SYN packet for port {}, ignoring", tcp_packet.get_destination());
|
trace!(?tcp_packet, "Received SYN packet for port {}, ignoring", tcp_packet.get_destination());
|
||||||
continue;
|
continue;
|
||||||
} else if (tcp_packet.get_flags() & tcp::TcpFlags::RST) == 0 {
|
} else if (tcp_packet.get_flags() & tcp::TcpFlags::RST) != 0 {
|
||||||
info!("Unknown RST TCP packet from {}, ignoring", remote_addr);
|
info!("Unknown RST TCP packet from {}, ignoring", remote_addr);
|
||||||
continue;
|
continue;
|
||||||
|
} else {
|
||||||
|
trace!("Unknown TCP packet from {}, ignoring", remote_addr);
|
||||||
|
continue;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
None => {
|
None => {
|
||||||
|
|||||||
@@ -730,6 +730,17 @@ impl ZCPacket {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
pub fn foreign_network_inner_packet_type(&self) -> Option<u8> {
|
||||||
|
if self.peer_manager_header()?.packet_type != PacketType::ForeignNetworkPacket as u8 {
|
||||||
|
return None;
|
||||||
|
}
|
||||||
|
|
||||||
|
let payload = self.payload();
|
||||||
|
let hdr = ForeignNetworkPacketHeader::ref_from_prefix(payload)?;
|
||||||
|
let inner_packet = payload.get(hdr.get_header_len()..)?;
|
||||||
|
PeerManagerHeader::ref_from_prefix(inner_packet).map(|hdr| hdr.packet_type)
|
||||||
|
}
|
||||||
|
|
||||||
pub fn foreign_network_packet(mut self) -> Self {
|
pub fn foreign_network_packet(mut self) -> Self {
|
||||||
let hdr = self.foreign_network_hdr().unwrap();
|
let hdr = self.foreign_network_hdr().unwrap();
|
||||||
let foreign_hdr_len = hdr.get_header_len();
|
let foreign_hdr_len = hdr.get_header_len();
|
||||||
|
|||||||
+201
-15
@@ -14,8 +14,8 @@ use derivative::Derivative;
|
|||||||
use derive_more::{Deref, DerefMut};
|
use derive_more::{Deref, DerefMut};
|
||||||
use parking_lot::RwLock;
|
use parking_lot::RwLock;
|
||||||
use quinn::{
|
use quinn::{
|
||||||
ClientConfig, Connection, Endpoint, EndpointConfig, ServerConfig, TransportConfig,
|
ClientConfig, ConnectError, Connection, Endpoint, EndpointConfig, ServerConfig,
|
||||||
congestion::BbrConfig, default_runtime,
|
TransportConfig, congestion::BbrConfig, default_runtime,
|
||||||
};
|
};
|
||||||
use std::net::{Ipv4Addr, Ipv6Addr};
|
use std::net::{Ipv4Addr, Ipv6Addr};
|
||||||
use std::sync::OnceLock;
|
use std::sync::OnceLock;
|
||||||
@@ -135,6 +135,12 @@ impl<Item> RwPool<Item> {
|
|||||||
self.resize();
|
self.resize();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn len(&self) -> usize {
|
||||||
|
let persistent_len = self.persistent.read().len();
|
||||||
|
let ephemeral_len = self.ephemeral.read().len();
|
||||||
|
persistent_len + ephemeral_len
|
||||||
|
}
|
||||||
|
|
||||||
/// try to push an item to the ephemeral pool, return the item if full
|
/// try to push an item to the ephemeral pool, return the item if full
|
||||||
fn try_push(&self, item: Item) -> Option<Item> {
|
fn try_push(&self, item: Item) -> Option<Item> {
|
||||||
let mut pool = self.ephemeral.write();
|
let mut pool = self.ephemeral.write();
|
||||||
@@ -168,6 +174,49 @@ impl<Item> RwPool<Item> {
|
|||||||
f(&mut persistent.iter().chain(ephemeral.iter()))
|
f(&mut persistent.iter().chain(ephemeral.iter()))
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
impl RwPool<Endpoint> {
|
||||||
|
fn retain_endpoints<F>(&self, mut keep: F) -> usize
|
||||||
|
where
|
||||||
|
F: FnMut(&Endpoint) -> bool,
|
||||||
|
{
|
||||||
|
let persistent_removed = {
|
||||||
|
let mut persistent = self.persistent.write();
|
||||||
|
let before = persistent.len();
|
||||||
|
persistent.retain(|endpoint| keep(endpoint));
|
||||||
|
before - persistent.len()
|
||||||
|
};
|
||||||
|
|
||||||
|
let ephemeral_removed = {
|
||||||
|
let mut ephemeral = self.ephemeral.write();
|
||||||
|
let before = ephemeral.len();
|
||||||
|
ephemeral.retain(|endpoint| keep(endpoint));
|
||||||
|
before - ephemeral.len()
|
||||||
|
};
|
||||||
|
|
||||||
|
let removed = persistent_removed + ephemeral_removed;
|
||||||
|
if removed > 0 {
|
||||||
|
self.resize();
|
||||||
|
}
|
||||||
|
removed
|
||||||
|
}
|
||||||
|
|
||||||
|
fn remove_by_local_addr(&self, local_addr: SocketAddr) -> usize {
|
||||||
|
self.retain_endpoints(|endpoint| endpoint.local_addr().ok() != Some(local_addr))
|
||||||
|
}
|
||||||
|
|
||||||
|
fn contains_local_addr(&self, local_addr: SocketAddr) -> bool {
|
||||||
|
self.persistent
|
||||||
|
.read()
|
||||||
|
.iter()
|
||||||
|
.any(|endpoint| endpoint.local_addr().ok() == Some(local_addr))
|
||||||
|
|| self
|
||||||
|
.ephemeral
|
||||||
|
.read()
|
||||||
|
.iter()
|
||||||
|
.any(|endpoint| endpoint.local_addr().ok() == Some(local_addr))
|
||||||
|
}
|
||||||
|
}
|
||||||
//endregion
|
//endregion
|
||||||
|
|
||||||
//region endpoint manager
|
//region endpoint manager
|
||||||
@@ -262,6 +311,20 @@ impl QuicEndpointManager {
|
|||||||
QUIC_ENDPOINT_MANAGER.get().unwrap()
|
QUIC_ENDPOINT_MANAGER.get().unwrap()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn client_pool(&self, ip_version: IpVersion) -> &RwPool<Endpoint> {
|
||||||
|
let dual_stack = self.both.is_enabled();
|
||||||
|
match ip_version {
|
||||||
|
IpVersion::V4 if !dual_stack => &self.ipv4,
|
||||||
|
_ => {
|
||||||
|
if dual_stack {
|
||||||
|
&self.both
|
||||||
|
} else {
|
||||||
|
&self.ipv6
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
/// Get a QUIC endpoint to be used as a server
|
/// Get a QUIC endpoint to be used as a server
|
||||||
///
|
///
|
||||||
/// # Arguments
|
/// # Arguments
|
||||||
@@ -288,14 +351,8 @@ impl QuicEndpointManager {
|
|||||||
Ok(endpoint)
|
Ok(endpoint)
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Get a quic endpoint to be used as a client
|
fn client_endpoint(&self, ip_version: IpVersion) -> Result<Endpoint, TunnelError> {
|
||||||
///
|
let (pool, endpoint) = self.create(|mgr| {
|
||||||
/// # Arguments
|
|
||||||
/// * `ip_version`: the IP version of the remote address
|
|
||||||
fn client(global_ctx: &ArcGlobalCtx, ip_version: IpVersion) -> Result<Endpoint, TunnelError> {
|
|
||||||
let mgr = Self::load(global_ctx);
|
|
||||||
|
|
||||||
let (pool, endpoint) = mgr.create(|mgr| {
|
|
||||||
let dual_stack = mgr.both.is_enabled();
|
let dual_stack = mgr.both.is_enabled();
|
||||||
let (pool, addr) = match ip_version {
|
let (pool, addr) = match ip_version {
|
||||||
IpVersion::V4 if !dual_stack => (&mgr.ipv4, (Ipv4Addr::UNSPECIFIED, 0).into()),
|
IpVersion::V4 if !dual_stack => (&mgr.ipv4, (Ipv4Addr::UNSPECIFIED, 0).into()),
|
||||||
@@ -318,6 +375,26 @@ impl QuicEndpointManager {
|
|||||||
Ok(pool.with_iter(|iter| iter.min_by_key(|e| e.open_connections()).unwrap().clone()))
|
Ok(pool.with_iter(|iter| iter.min_by_key(|e| e.open_connections()).unwrap().clone()))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn remove_endpoint(&self, endpoint: &Endpoint) -> usize {
|
||||||
|
let Ok(local_addr) = endpoint.local_addr() else {
|
||||||
|
return 0;
|
||||||
|
};
|
||||||
|
self.remove_endpoint_by_local_addr(local_addr)
|
||||||
|
}
|
||||||
|
|
||||||
|
fn remove_endpoint_by_local_addr(&self, local_addr: SocketAddr) -> usize {
|
||||||
|
[&self.ipv4, &self.ipv6, &self.both]
|
||||||
|
.into_iter()
|
||||||
|
.map(|pool| pool.remove_by_local_addr(local_addr))
|
||||||
|
.sum()
|
||||||
|
}
|
||||||
|
|
||||||
|
fn contains_local_addr(&self, local_addr: SocketAddr) -> bool {
|
||||||
|
[&self.ipv4, &self.ipv6, &self.both]
|
||||||
|
.into_iter()
|
||||||
|
.any(|pool| pool.contains_local_addr(local_addr))
|
||||||
|
}
|
||||||
|
|
||||||
async fn connect(
|
async fn connect(
|
||||||
global_ctx: &ArcGlobalCtx,
|
global_ctx: &ArcGlobalCtx,
|
||||||
addr: SocketAddr,
|
addr: SocketAddr,
|
||||||
@@ -327,14 +404,52 @@ impl QuicEndpointManager {
|
|||||||
} else {
|
} else {
|
||||||
IpVersion::V6
|
IpVersion::V6
|
||||||
};
|
};
|
||||||
let endpoint = Self::client(global_ctx, ip_version)?;
|
Self::load(global_ctx)
|
||||||
let connection = endpoint
|
.connect_with_ip_version(addr, ip_version)
|
||||||
.connect(addr, "localhost")
|
.await
|
||||||
.with_context(|| format!("failed to create connection to {}", addr))?
|
}
|
||||||
|
|
||||||
|
async fn connect_with_ip_version(
|
||||||
|
&self,
|
||||||
|
addr: SocketAddr,
|
||||||
|
ip_version: IpVersion,
|
||||||
|
) -> Result<(Endpoint, Connection), TunnelError> {
|
||||||
|
let max_endpoint_stopping_retries = self.client_pool(ip_version).len().saturating_add(1);
|
||||||
|
let mut endpoint_stopping_retries = 0;
|
||||||
|
|
||||||
|
loop {
|
||||||
|
let endpoint = self.client_endpoint(ip_version)?;
|
||||||
|
let connecting = match endpoint.connect(addr, "localhost") {
|
||||||
|
Ok(connecting) => connecting,
|
||||||
|
Err(ConnectError::EndpointStopping) => {
|
||||||
|
let local_addr = endpoint.local_addr().ok();
|
||||||
|
let removed = self.remove_endpoint(&endpoint);
|
||||||
|
endpoint_stopping_retries += 1;
|
||||||
|
tracing::warn!(
|
||||||
|
?addr,
|
||||||
|
?local_addr,
|
||||||
|
removed,
|
||||||
|
"removed stopped quic endpoint and retry connect"
|
||||||
|
);
|
||||||
|
if endpoint_stopping_retries > max_endpoint_stopping_retries {
|
||||||
|
return Err(anyhow::Error::new(ConnectError::EndpointStopping)
|
||||||
|
.context(format!("failed to create connection to {}", addr))
|
||||||
|
.into());
|
||||||
|
}
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
Err(e) => {
|
||||||
|
return Err(anyhow::Error::new(e)
|
||||||
|
.context(format!("failed to create connection to {}", addr))
|
||||||
|
.into());
|
||||||
|
}
|
||||||
|
};
|
||||||
|
let connection = connecting
|
||||||
.await
|
.await
|
||||||
.with_context(|| format!("failed to connect to {}", addr))?;
|
.with_context(|| format!("failed to connect to {}", addr))?;
|
||||||
|
|
||||||
Ok((endpoint, connection))
|
return Ok((endpoint, connection));
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
//endregion
|
//endregion
|
||||||
@@ -398,6 +513,18 @@ impl QuicTunnelListener {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
impl Drop for QuicTunnelListener {
|
||||||
|
fn drop(&mut self) {
|
||||||
|
let Some(endpoint) = &self.endpoint else {
|
||||||
|
return;
|
||||||
|
};
|
||||||
|
let Ok(local_addr) = endpoint.local_addr() else {
|
||||||
|
return;
|
||||||
|
};
|
||||||
|
QuicEndpointManager::load(&self.global_ctx).remove_endpoint_by_local_addr(local_addr);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
#[async_trait::async_trait]
|
#[async_trait::async_trait]
|
||||||
impl TunnelListener for QuicTunnelListener {
|
impl TunnelListener for QuicTunnelListener {
|
||||||
async fn listen(&mut self) -> Result<(), TunnelError> {
|
async fn listen(&mut self) -> Result<(), TunnelError> {
|
||||||
@@ -516,6 +643,20 @@ mod tests {
|
|||||||
get_mock_global_ctx_with_network(Some(identity))
|
get_mock_global_ctx_with_network(Some(identity))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn stopped_client_endpoint() -> (Endpoint, SocketAddr) {
|
||||||
|
let rt = Builder::new_current_thread().enable_all().build().unwrap();
|
||||||
|
let endpoint = rt.block_on(async {
|
||||||
|
QuicEndpointManager::try_create((Ipv4Addr::UNSPECIFIED, 0).into(), false).unwrap()
|
||||||
|
});
|
||||||
|
let local_addr = endpoint.local_addr().unwrap();
|
||||||
|
drop(rt);
|
||||||
|
assert!(matches!(
|
||||||
|
endpoint.connect("127.0.0.1:1".parse().unwrap(), "localhost"),
|
||||||
|
Err(ConnectError::EndpointStopping)
|
||||||
|
));
|
||||||
|
(endpoint, local_addr)
|
||||||
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
fn quic_pingpong() {
|
fn quic_pingpong() {
|
||||||
RUNTIME.block_on(quic_pingpong_impl())
|
RUNTIME.block_on(quic_pingpong_impl())
|
||||||
@@ -591,6 +732,51 @@ mod tests {
|
|||||||
assert!(port > 0);
|
assert!(port > 0);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn listener_drop_removes_persistent_endpoint() {
|
||||||
|
RUNTIME.block_on(listener_drop_removes_persistent_endpoint_impl())
|
||||||
|
}
|
||||||
|
async fn listener_drop_removes_persistent_endpoint_impl() {
|
||||||
|
let global_ctx = global_ctx();
|
||||||
|
let endpoint_addr = {
|
||||||
|
let mut listener =
|
||||||
|
QuicTunnelListener::new("quic://127.0.0.1:0".parse().unwrap(), global_ctx.clone());
|
||||||
|
listener.listen().await.unwrap();
|
||||||
|
let endpoint_addr = listener.endpoint.as_ref().unwrap().local_addr().unwrap();
|
||||||
|
assert!(QuicEndpointManager::load(&global_ctx).contains_local_addr(endpoint_addr));
|
||||||
|
endpoint_addr
|
||||||
|
};
|
||||||
|
|
||||||
|
assert!(!QuicEndpointManager::load(&global_ctx).contains_local_addr(endpoint_addr));
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn connect_removes_stopped_endpoints_and_retries() {
|
||||||
|
let (stopped_endpoint_a, stopped_addr_a) = stopped_client_endpoint();
|
||||||
|
let (stopped_endpoint_b, stopped_addr_b) = stopped_client_endpoint();
|
||||||
|
|
||||||
|
RUNTIME.block_on(async move {
|
||||||
|
let mgr = QuicEndpointManager::new(2);
|
||||||
|
mgr.both.push(stopped_endpoint_a);
|
||||||
|
mgr.both.push(stopped_endpoint_b);
|
||||||
|
assert!(mgr.contains_local_addr(stopped_addr_a));
|
||||||
|
assert!(mgr.contains_local_addr(stopped_addr_b));
|
||||||
|
|
||||||
|
let err = mgr
|
||||||
|
.connect_with_ip_version("127.0.0.1:0".parse().unwrap(), IpVersion::V4)
|
||||||
|
.await
|
||||||
|
.unwrap_err();
|
||||||
|
let err = format!("{:?}", err);
|
||||||
|
assert!(
|
||||||
|
err.contains("invalid remote address"),
|
||||||
|
"unexpected error: {}",
|
||||||
|
err
|
||||||
|
);
|
||||||
|
assert!(!mgr.contains_local_addr(stopped_addr_a));
|
||||||
|
assert!(!mgr.contains_local_addr(stopped_addr_b));
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
fn invalid_peer_addr() {
|
fn invalid_peer_addr() {
|
||||||
RUNTIME.block_on(invalid_peer_addr_impl())
|
RUNTIME.block_on(invalid_peer_addr_impl())
|
||||||
|
|||||||
@@ -2,13 +2,17 @@ use std::sync::Arc;
|
|||||||
|
|
||||||
use crate::{
|
use crate::{
|
||||||
common::{
|
common::{
|
||||||
config::TomlConfigLoader, global_ctx::GlobalCtx, log, os_info::collect_device_os_info,
|
config::TomlConfigLoader,
|
||||||
set_default_machine_id, stun::MockStunInfoCollector,
|
global_ctx::{ArcGlobalCtx, GlobalCtx},
|
||||||
|
log,
|
||||||
|
os_info::collect_device_os_info,
|
||||||
|
set_default_machine_id,
|
||||||
|
stun::MockStunInfoCollector,
|
||||||
},
|
},
|
||||||
connector::create_connector_by_url,
|
connector::create_connector_by_url,
|
||||||
instance_manager::{DaemonGuard, NetworkInstanceManager},
|
instance_manager::{DaemonGuard, NetworkInstanceManager},
|
||||||
proto::common::NatType,
|
proto::common::NatType,
|
||||||
tunnel::{IpVersion, TunnelConnector},
|
tunnel::{IpVersion, Tunnel, TunnelConnector, TunnelError, TunnelScheme},
|
||||||
};
|
};
|
||||||
use anyhow::{Context as _, Result};
|
use anyhow::{Context as _, Result};
|
||||||
use async_trait::async_trait;
|
use async_trait::async_trait;
|
||||||
@@ -49,6 +53,30 @@ pub struct WebClient {
|
|||||||
connected: Arc<AtomicBool>,
|
connected: Arc<AtomicBool>,
|
||||||
}
|
}
|
||||||
|
|
||||||
|
struct ConfigServerConnector {
|
||||||
|
url: Url,
|
||||||
|
global_ctx: ArcGlobalCtx,
|
||||||
|
}
|
||||||
|
|
||||||
|
#[async_trait]
|
||||||
|
impl TunnelConnector for ConfigServerConnector {
|
||||||
|
async fn connect(&mut self) -> std::result::Result<Box<dyn Tunnel>, TunnelError> {
|
||||||
|
let mut connector =
|
||||||
|
create_connector_by_url(self.url.as_str(), &self.global_ctx, IpVersion::Both)
|
||||||
|
.await
|
||||||
|
.map_err(|err| match err {
|
||||||
|
crate::common::error::Error::TunnelError(err) => err,
|
||||||
|
err => TunnelError::Anyhow(err.into()),
|
||||||
|
})?;
|
||||||
|
|
||||||
|
connector.connect().await
|
||||||
|
}
|
||||||
|
|
||||||
|
fn remote_url(&self) -> Url {
|
||||||
|
self.url.clone()
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
impl WebClient {
|
impl WebClient {
|
||||||
pub fn new<T: TunnelConnector + 'static, S: ToString, H: ToString>(
|
pub fn new<T: TunnelConnector + 'static, S: ToString, H: ToString>(
|
||||||
connector: T,
|
connector: T,
|
||||||
@@ -218,6 +246,13 @@ pub async fn run_web_client(
|
|||||||
.with_context(|| "failed to parse config server URL")?,
|
.with_context(|| "failed to parse config server URL")?,
|
||||||
};
|
};
|
||||||
|
|
||||||
|
TunnelScheme::try_from(&config_server_url).map_err(|_| {
|
||||||
|
anyhow::anyhow!(
|
||||||
|
"unsupported config server scheme: {}",
|
||||||
|
config_server_url.scheme()
|
||||||
|
)
|
||||||
|
})?;
|
||||||
|
|
||||||
let mut c_url = config_server_url.clone();
|
let mut c_url = config_server_url.clone();
|
||||||
if !matches!(c_url.scheme(), "ws" | "wss") {
|
if !matches!(c_url.scheme(), "ws" | "wss") {
|
||||||
c_url.set_path("");
|
c_url.set_path("");
|
||||||
@@ -243,16 +278,20 @@ pub async fn run_web_client(
|
|||||||
let mut flags = global_ctx.get_flags();
|
let mut flags = global_ctx.get_flags();
|
||||||
flags.bind_device = false;
|
flags.bind_device = false;
|
||||||
global_ctx.set_flags(flags);
|
global_ctx.set_flags(flags);
|
||||||
|
|
||||||
let hostname = match hostname {
|
let hostname = match hostname {
|
||||||
None => gethostname::gethostname().to_string_lossy().to_string(),
|
None => gethostname::gethostname().to_string_lossy().to_string(),
|
||||||
Some(hostname) => hostname,
|
Some(hostname) => hostname,
|
||||||
};
|
};
|
||||||
Ok(WebClient::new(
|
Ok(WebClient::new(
|
||||||
create_connector_by_url(c_url.as_str(), &global_ctx, IpVersion::Both).await?,
|
ConfigServerConnector {
|
||||||
|
url: c_url,
|
||||||
|
global_ctx,
|
||||||
|
},
|
||||||
token.to_string(),
|
token.to_string(),
|
||||||
hostname,
|
hostname,
|
||||||
secure_mode,
|
secure_mode,
|
||||||
manager.clone(),
|
manager,
|
||||||
hooks,
|
hooks,
|
||||||
))
|
))
|
||||||
}
|
}
|
||||||
@@ -292,4 +331,23 @@ mod tests {
|
|||||||
assert!(sleep_finish.load(std::sync::atomic::Ordering::Relaxed));
|
assert!(sleep_finish.load(std::sync::atomic::Ordering::Relaxed));
|
||||||
println!("Manager stopped.");
|
println!("Manager stopped.");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn test_run_web_client_with_unreachable_config_server() {
|
||||||
|
let manager = Arc::new(NetworkInstanceManager::new());
|
||||||
|
let client = super::run_web_client(
|
||||||
|
"udp://config-server.invalid:22020/test",
|
||||||
|
None,
|
||||||
|
None,
|
||||||
|
false,
|
||||||
|
manager,
|
||||||
|
None,
|
||||||
|
)
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
|
||||||
|
tokio::time::sleep(std::time::Duration::from_millis(100)).await;
|
||||||
|
assert!(!client.is_connected());
|
||||||
|
drop(client);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user