Compare commits
2 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| a87e9578eb | |||
| b851bc7994 |
@@ -1,5 +1,11 @@
|
|||||||
# Changelog
|
# Changelog
|
||||||
|
|
||||||
|
## 2026-03-17 - 4.7.2 - fix(remoteingress-core)
|
||||||
|
add tunnel write timeouts and scale initial stream windows by active stream count
|
||||||
|
|
||||||
|
- Wrap tunnel frame writes and flushes in a 30-second timeout on both edge and hub to detect stalled writers and trigger faster reconnect or cleanup.
|
||||||
|
- Compute each stream's initial send window from the current active stream count instead of using a fixed window to keep total in-flight data within the 32MB budget.
|
||||||
|
|
||||||
## 2026-03-17 - 4.7.1 - fix(remoteingress-core)
|
## 2026-03-17 - 4.7.1 - fix(remoteingress-core)
|
||||||
improve tunnel failure detection and reconnect handling
|
improve tunnel failure detection and reconnect handling
|
||||||
|
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "@serve.zone/remoteingress",
|
"name": "@serve.zone/remoteingress",
|
||||||
"version": "4.7.1",
|
"version": "4.7.2",
|
||||||
"private": false,
|
"private": false,
|
||||||
"description": "Edge ingress tunnel for DcRouter - accepts incoming TCP connections at network edge and tunnels them to DcRouter SmartProxy preserving client IP via PROXY protocol v1.",
|
"description": "Edge ingress tunnel for DcRouter - accepts incoming TCP connections at network edge and tunnels them to DcRouter SmartProxy preserving client IP via PROXY protocol v1.",
|
||||||
"main": "dist_ts/index.js",
|
"main": "dist_ts/index.js",
|
||||||
|
|||||||
@@ -403,14 +403,18 @@ async fn connect_to_hub_and_run(
|
|||||||
// TLS records and syscalls. Flushed after each frame to avoid holding data.
|
// TLS records and syscalls. Flushed after each frame to avoid holding data.
|
||||||
let mut writer = tokio::io::BufWriter::with_capacity(65536, write_half);
|
let mut writer = tokio::io::BufWriter::with_capacity(65536, write_half);
|
||||||
let mut write_error = false;
|
let mut write_error = false;
|
||||||
|
let write_timeout = Duration::from_secs(30);
|
||||||
loop {
|
loop {
|
||||||
tokio::select! {
|
tokio::select! {
|
||||||
biased; // control frames always take priority over data
|
biased; // control frames always take priority over data
|
||||||
ctrl = tunnel_ctrl_rx.recv() => {
|
ctrl = tunnel_ctrl_rx.recv() => {
|
||||||
match ctrl {
|
match ctrl {
|
||||||
Some(frame_data) => {
|
Some(frame_data) => {
|
||||||
if writer.write_all(&frame_data).await.is_err() { write_error = true; break; }
|
let ok = tokio::time::timeout(write_timeout, async {
|
||||||
if writer.flush().await.is_err() { write_error = true; break; }
|
writer.write_all(&frame_data).await?;
|
||||||
|
writer.flush().await
|
||||||
|
}).await;
|
||||||
|
if !matches!(ok, Ok(Ok(()))) { write_error = true; break; }
|
||||||
}
|
}
|
||||||
None => break,
|
None => break,
|
||||||
}
|
}
|
||||||
@@ -418,8 +422,11 @@ async fn connect_to_hub_and_run(
|
|||||||
data = tunnel_data_rx.recv() => {
|
data = tunnel_data_rx.recv() => {
|
||||||
match data {
|
match data {
|
||||||
Some(frame_data) => {
|
Some(frame_data) => {
|
||||||
if writer.write_all(&frame_data).await.is_err() { write_error = true; break; }
|
let ok = tokio::time::timeout(write_timeout, async {
|
||||||
if writer.flush().await.is_err() { write_error = true; break; }
|
writer.write_all(&frame_data).await?;
|
||||||
|
writer.flush().await
|
||||||
|
}).await;
|
||||||
|
if !matches!(ok, Ok(Ok(()))) { write_error = true; break; }
|
||||||
}
|
}
|
||||||
None => break,
|
None => break,
|
||||||
}
|
}
|
||||||
@@ -428,7 +435,7 @@ async fn connect_to_hub_and_run(
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
if write_error {
|
if write_error {
|
||||||
log::error!("Tunnel writer failed, signalling reader for fast reconnect");
|
log::error!("Tunnel writer failed or stalled, signalling reader for fast reconnect");
|
||||||
let _ = writer_dead_tx.send(());
|
let _ = writer_dead_tx.send(());
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
@@ -712,7 +719,12 @@ async fn handle_client_connection(
|
|||||||
|
|
||||||
// Set up channel for data coming back from hub (capacity 16 is sufficient with flow control)
|
// Set up channel for data coming back from hub (capacity 16 is sufficient with flow control)
|
||||||
let (back_tx, mut back_rx) = mpsc::channel::<Vec<u8>>(256);
|
let (back_tx, mut back_rx) = mpsc::channel::<Vec<u8>>(256);
|
||||||
let send_window = Arc::new(AtomicU32::new(INITIAL_STREAM_WINDOW));
|
// Adaptive initial window: scale with current stream count to keep total in-flight
|
||||||
|
// data within the 32MB budget. Prevents burst flooding when many streams open.
|
||||||
|
let initial_window = remoteingress_protocol::compute_window_for_stream_count(
|
||||||
|
active_streams.load(Ordering::Relaxed),
|
||||||
|
);
|
||||||
|
let send_window = Arc::new(AtomicU32::new(initial_window));
|
||||||
let window_notify = Arc::new(Notify::new());
|
let window_notify = Arc::new(Notify::new());
|
||||||
{
|
{
|
||||||
let mut writers = client_writers.lock().await;
|
let mut writers = client_writers.lock().await;
|
||||||
|
|||||||
@@ -396,14 +396,18 @@ async fn handle_edge_connection(
|
|||||||
// TLS records and syscalls. Flushed after each frame to avoid holding data.
|
// TLS records and syscalls. Flushed after each frame to avoid holding data.
|
||||||
let mut writer = tokio::io::BufWriter::with_capacity(65536, write_half);
|
let mut writer = tokio::io::BufWriter::with_capacity(65536, write_half);
|
||||||
let mut write_error = false;
|
let mut write_error = false;
|
||||||
|
let write_timeout = Duration::from_secs(30);
|
||||||
loop {
|
loop {
|
||||||
tokio::select! {
|
tokio::select! {
|
||||||
biased; // control frames always take priority over data
|
biased; // control frames always take priority over data
|
||||||
ctrl = ctrl_rx.recv() => {
|
ctrl = ctrl_rx.recv() => {
|
||||||
match ctrl {
|
match ctrl {
|
||||||
Some(frame_data) => {
|
Some(frame_data) => {
|
||||||
if writer.write_all(&frame_data).await.is_err() { write_error = true; break; }
|
let ok = tokio::time::timeout(write_timeout, async {
|
||||||
if writer.flush().await.is_err() { write_error = true; break; }
|
writer.write_all(&frame_data).await?;
|
||||||
|
writer.flush().await
|
||||||
|
}).await;
|
||||||
|
if !matches!(ok, Ok(Ok(()))) { write_error = true; break; }
|
||||||
}
|
}
|
||||||
None => break,
|
None => break,
|
||||||
}
|
}
|
||||||
@@ -411,8 +415,11 @@ async fn handle_edge_connection(
|
|||||||
data = data_rx.recv() => {
|
data = data_rx.recv() => {
|
||||||
match data {
|
match data {
|
||||||
Some(frame_data) => {
|
Some(frame_data) => {
|
||||||
if writer.write_all(&frame_data).await.is_err() { write_error = true; break; }
|
let ok = tokio::time::timeout(write_timeout, async {
|
||||||
if writer.flush().await.is_err() { write_error = true; break; }
|
writer.write_all(&frame_data).await?;
|
||||||
|
writer.flush().await
|
||||||
|
}).await;
|
||||||
|
if !matches!(ok, Ok(Ok(()))) { write_error = true; break; }
|
||||||
}
|
}
|
||||||
None => break,
|
None => break,
|
||||||
}
|
}
|
||||||
@@ -421,7 +428,7 @@ async fn handle_edge_connection(
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
if write_error {
|
if write_error {
|
||||||
log::error!("Tunnel writer to edge failed, signalling reader for fast cleanup");
|
log::error!("Tunnel writer to edge failed or stalled, signalling reader for fast cleanup");
|
||||||
let _ = writer_dead_tx.send(());
|
let _ = writer_dead_tx.send(());
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
@@ -512,7 +519,12 @@ async fn handle_edge_connection(
|
|||||||
|
|
||||||
// Create channel for data from edge to this stream (capacity 16 is sufficient with flow control)
|
// Create channel for data from edge to this stream (capacity 16 is sufficient with flow control)
|
||||||
let (data_tx, mut data_rx) = mpsc::channel::<Vec<u8>>(256);
|
let (data_tx, mut data_rx) = mpsc::channel::<Vec<u8>>(256);
|
||||||
let send_window = Arc::new(AtomicU32::new(INITIAL_STREAM_WINDOW));
|
// Adaptive initial window: scale with current stream count
|
||||||
|
// to keep total in-flight data within the 32MB budget.
|
||||||
|
let initial_window = compute_window_for_stream_count(
|
||||||
|
edge_stream_count.load(Ordering::Relaxed),
|
||||||
|
);
|
||||||
|
let send_window = Arc::new(AtomicU32::new(initial_window));
|
||||||
let window_notify = Arc::new(Notify::new());
|
let window_notify = Arc::new(Notify::new());
|
||||||
{
|
{
|
||||||
let mut s = streams.lock().await;
|
let mut s = streams.lock().await;
|
||||||
|
|||||||
@@ -3,6 +3,6 @@
|
|||||||
*/
|
*/
|
||||||
export const commitinfo = {
|
export const commitinfo = {
|
||||||
name: '@serve.zone/remoteingress',
|
name: '@serve.zone/remoteingress',
|
||||||
version: '4.7.1',
|
version: '4.7.2',
|
||||||
description: 'Edge ingress tunnel for DcRouter - accepts incoming TCP connections at network edge and tunnels them to DcRouter SmartProxy preserving client IP via PROXY protocol v1.'
|
description: 'Edge ingress tunnel for DcRouter - accepts incoming TCP connections at network edge and tunnels them to DcRouter SmartProxy preserving client IP via PROXY protocol v1.'
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user