Serverless listener forwards /start to the application when basePath is "/"

Abierto Apto para principiantes
  • ¿Algo no está bien?
#5,755 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

@Saisharathchandranandnetha ya está trabajando en esto.

Desde el 19/9/2026.

  • #5756 de @Saisharathchandranandnetha — abierto

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
90/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
rust
Área
backend

Línea de trabajo

Comienza en rivetkit-rust/packages/rivetkit-core/src/serverless.rs, en handles_listener_request; después revisa route_path y las pruebas serverless relacionadas. Ejecuta el cargo test indicado con la reproducción de la ruta base raíz; se considera terminado cuando /, /start, /health, /metadata y /metrics están reservados, mientras que /application se reenvía a la aplicación.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

What happens

With serverless.basePath: "/" and an application passed to registry.listen(), the Rust listener forwards the engine's POST /start to the application instead of handling it. Same for /health, /metadata and /metrics. Actors on that pool never start. The same config works through registry.handler().

Where

handles_listener_request in rivetkit-rust/packages/rivetkit-core/src/serverless.rs:

let request_path = parsed.path();
if request_path != base_path && !request_path.starts_with(&format!("{base_path}/")) {
	return false;
}

Why it happens

normalize_base_path(Some("/")) returns "/", so the prefix check looks for "//". No request path starts with //, so everything except / itself returns false. serverless_http.rs only hands a request to the runtime when this returns true or when there's no application, so /start ends up in the user's handler.

route_path already handles a root base path correctly. It's only this check. The TypeScript side special-cases it in isServerlessStartRequest (basePath === "/" ? "" : ...), which is why registry.handler() works.

Reproduction

Add to rivetkit-rust/packages/rivetkit-core/tests/serverless.rs:

#[test]
fn listener_reserves_framework_routes_for_root_base_path() {
	let base_path = normalize_base_path(Some("/"));
	for path in ["/", "/start", "/health", "/metadata", "/metrics"] {
		assert!(
			handles_listener_request(&base_path, &format!("http://internal{path}")),
			"root base path should reserve {path}"
		);
	}
	assert!(!handles_listener_request(&base_path, "http://internal/application"));
}

cargo test -p rivetkit-core --lib listener_reserves_framework_routes_for_root_base_path fails with root base path should reserve /start.

Expected behaviour

With a root base path the listener handles /, /start, /health, /metadata and /metrics itself and forwards everything else to the application.

Suggested fix

Treat base_path == "/" as matching every request path before the prefix check. route_path already returns the right route in that case. Happy to open a PR.

Environment

  • upstream/main at 78336a1a0
  • Linux 7.2.5-3-omarchy
  • rustc 1.98.1 (48a229cea 2026-09-01), cargo 1.98.1 (797e8a9bc 2026-08-05)
Lenguaje dominante
Rust
Estrellas
6.6k
Forks
304
Merge medio
18 h 5 min
PR fusionados (30 d)
31

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de rivet-dev/rivet

Todos los issues de rivet-dev/rivet

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.