mirror of
https://github.com/openai/codex.git
synced 2026-05-24 04:54:52 +00:00
## Summary
- migrate exec-server remote registration naming from executor to
environment
- align CLI, public Rust exports, registry error messages, and relay
test fixtures with the environment registry contract
- keep the live registration path and response model consistent with
`/cloud/environment/{environment_id}/register`
## Verification
- `cargo test -p codex-exec-server
remote::tests::register_environment_posts_with_auth_provider_headers
--manifest-path /Users/richardlee/code/codex/codex-rs/Cargo.toml`
- `cargo test -p codex-exec-server --test relay
multiplexed_remote_environment_routes_independent_virtual_streams
--manifest-path /Users/richardlee/code/codex/codex-rs/Cargo.toml`
- `cargo check -p codex-cli --manifest-path
/Users/richardlee/code/codex/codex-rs/Cargo.toml` (still running when PR
opened; will update after completion if needed)
330 lines
10 KiB
Rust
330 lines
10 KiB
Rust
use std::time::Duration;
|
|
|
|
use codex_api::SharedAuthProvider;
|
|
use reqwest::StatusCode;
|
|
use serde::Deserialize;
|
|
use tokio::time::sleep;
|
|
use tokio_tungstenite::connect_async;
|
|
use tracing::warn;
|
|
|
|
use codex_utils_rustls_provider::ensure_rustls_crypto_provider;
|
|
|
|
use crate::ExecServerError;
|
|
use crate::ExecServerRuntimePaths;
|
|
use crate::relay::run_multiplexed_environment;
|
|
use crate::server::ConnectionProcessor;
|
|
|
|
const ERROR_BODY_PREVIEW_BYTES: usize = 4096;
|
|
|
|
#[derive(Clone)]
|
|
struct EnvironmentRegistryClient {
|
|
base_url: String,
|
|
auth_provider: SharedAuthProvider,
|
|
http: reqwest::Client,
|
|
}
|
|
|
|
impl std::fmt::Debug for EnvironmentRegistryClient {
|
|
fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
|
|
f.debug_struct("EnvironmentRegistryClient")
|
|
.field("base_url", &self.base_url)
|
|
.field("auth_provider", &"<redacted>")
|
|
.finish_non_exhaustive()
|
|
}
|
|
}
|
|
|
|
impl EnvironmentRegistryClient {
|
|
fn new(base_url: String, auth_provider: SharedAuthProvider) -> Result<Self, ExecServerError> {
|
|
let base_url = normalize_base_url(base_url)?;
|
|
Ok(Self {
|
|
base_url,
|
|
auth_provider,
|
|
http: reqwest::Client::new(),
|
|
})
|
|
}
|
|
|
|
async fn register_environment(
|
|
&self,
|
|
environment_id: &str,
|
|
) -> Result<EnvironmentRegistryRegistrationResponse, ExecServerError> {
|
|
let response = self
|
|
.http
|
|
.post(endpoint_url(
|
|
&self.base_url,
|
|
&format!("/cloud/environment/{environment_id}/register"),
|
|
))
|
|
.headers(self.auth_provider.to_auth_headers())
|
|
.send()
|
|
.await?;
|
|
self.parse_json_response(response).await
|
|
}
|
|
|
|
async fn parse_json_response<R>(
|
|
&self,
|
|
response: reqwest::Response,
|
|
) -> Result<R, ExecServerError>
|
|
where
|
|
R: for<'de> Deserialize<'de>,
|
|
{
|
|
if response.status().is_success() {
|
|
return response.json::<R>().await.map_err(ExecServerError::from);
|
|
}
|
|
|
|
let status = response.status();
|
|
let body = response.text().await.unwrap_or_default();
|
|
if matches!(status, StatusCode::UNAUTHORIZED | StatusCode::FORBIDDEN) {
|
|
return Err(environment_registry_auth_error(status, &body));
|
|
}
|
|
|
|
Err(environment_registry_http_error(status, &body))
|
|
}
|
|
}
|
|
|
|
#[derive(Debug, Clone, Eq, PartialEq, Deserialize)]
|
|
struct EnvironmentRegistryRegistrationResponse {
|
|
environment_id: String,
|
|
url: String,
|
|
}
|
|
|
|
/// Configuration for registering an exec-server for remote use.
|
|
#[derive(Clone)]
|
|
pub struct RemoteEnvironmentConfig {
|
|
pub base_url: String,
|
|
pub environment_id: String,
|
|
pub name: String,
|
|
auth_provider: SharedAuthProvider,
|
|
}
|
|
|
|
impl std::fmt::Debug for RemoteEnvironmentConfig {
|
|
fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
|
|
f.debug_struct("RemoteEnvironmentConfig")
|
|
.field("base_url", &self.base_url)
|
|
.field("environment_id", &self.environment_id)
|
|
.field("name", &self.name)
|
|
.field("auth_provider", &"<redacted>")
|
|
.finish()
|
|
}
|
|
}
|
|
|
|
impl RemoteEnvironmentConfig {
|
|
pub fn new(
|
|
base_url: String,
|
|
environment_id: String,
|
|
auth_provider: SharedAuthProvider,
|
|
) -> Result<Self, ExecServerError> {
|
|
let environment_id = normalize_environment_id(environment_id)?;
|
|
Ok(Self {
|
|
base_url,
|
|
environment_id,
|
|
name: "codex-exec-server".to_string(),
|
|
auth_provider,
|
|
})
|
|
}
|
|
}
|
|
|
|
/// Register an exec-server for remote use and serve requests over the returned
|
|
/// rendezvous websocket.
|
|
pub async fn run_remote_environment(
|
|
config: RemoteEnvironmentConfig,
|
|
runtime_paths: ExecServerRuntimePaths,
|
|
) -> Result<(), ExecServerError> {
|
|
ensure_rustls_crypto_provider();
|
|
let client =
|
|
EnvironmentRegistryClient::new(config.base_url.clone(), config.auth_provider.clone())?;
|
|
let processor = ConnectionProcessor::new(runtime_paths);
|
|
let mut backoff = Duration::from_secs(1);
|
|
|
|
loop {
|
|
let response = client.register_environment(&config.environment_id).await?;
|
|
eprintln!(
|
|
"codex exec-server remote environment registered with environment_id {}",
|
|
response.environment_id
|
|
);
|
|
|
|
match connect_async(response.url.as_str()).await {
|
|
Ok((websocket, _)) => {
|
|
backoff = Duration::from_secs(1);
|
|
run_multiplexed_environment(websocket, processor.clone()).await;
|
|
}
|
|
Err(err) => {
|
|
warn!("failed to connect remote exec-server websocket: {err}");
|
|
}
|
|
}
|
|
|
|
sleep(backoff).await;
|
|
backoff = (backoff * 2).min(Duration::from_secs(30));
|
|
}
|
|
}
|
|
|
|
fn normalize_environment_id(environment_id: String) -> Result<String, ExecServerError> {
|
|
let environment_id = environment_id.trim().to_string();
|
|
if environment_id.is_empty() {
|
|
return Err(ExecServerError::EnvironmentRegistryConfig(
|
|
"environment id is required for remote exec-server registration".to_string(),
|
|
));
|
|
}
|
|
Ok(environment_id)
|
|
}
|
|
|
|
#[derive(Deserialize)]
|
|
struct RegistryErrorBody {
|
|
error: Option<RegistryError>,
|
|
}
|
|
|
|
#[derive(Deserialize)]
|
|
struct RegistryError {
|
|
code: Option<String>,
|
|
message: Option<String>,
|
|
}
|
|
|
|
fn normalize_base_url(base_url: String) -> Result<String, ExecServerError> {
|
|
let trimmed = base_url.trim().trim_end_matches('/').to_string();
|
|
if trimmed.is_empty() {
|
|
return Err(ExecServerError::EnvironmentRegistryConfig(
|
|
"environment registry base URL is required".to_string(),
|
|
));
|
|
}
|
|
Ok(trimmed)
|
|
}
|
|
|
|
fn endpoint_url(base_url: &str, path: &str) -> String {
|
|
format!("{base_url}/{}", path.trim_start_matches('/'))
|
|
}
|
|
|
|
fn environment_registry_auth_error(status: StatusCode, body: &str) -> ExecServerError {
|
|
let message = registry_error_message(body).unwrap_or_else(|| "empty error body".to_string());
|
|
ExecServerError::EnvironmentRegistryAuth(format!(
|
|
"environment registry authentication failed ({status}): {message}"
|
|
))
|
|
}
|
|
|
|
fn environment_registry_http_error(status: StatusCode, body: &str) -> ExecServerError {
|
|
let parsed = serde_json::from_str::<RegistryErrorBody>(body).ok();
|
|
let (code, message) = parsed
|
|
.and_then(|body| body.error)
|
|
.map(|error| {
|
|
(
|
|
error.code,
|
|
error.message.unwrap_or_else(|| {
|
|
preview_error_body(body).unwrap_or_else(|| "empty error body".to_string())
|
|
}),
|
|
)
|
|
})
|
|
.unwrap_or_else(|| {
|
|
(
|
|
None,
|
|
preview_error_body(body)
|
|
.unwrap_or_else(|| "empty or malformed error body".to_string()),
|
|
)
|
|
});
|
|
ExecServerError::EnvironmentRegistryHttp {
|
|
status,
|
|
code,
|
|
message,
|
|
}
|
|
}
|
|
|
|
fn registry_error_message(body: &str) -> Option<String> {
|
|
serde_json::from_str::<RegistryErrorBody>(body)
|
|
.ok()
|
|
.and_then(|body| body.error)
|
|
.and_then(|error| error.message)
|
|
.or_else(|| preview_error_body(body))
|
|
}
|
|
|
|
fn preview_error_body(body: &str) -> Option<String> {
|
|
let trimmed = body.trim();
|
|
if trimmed.is_empty() {
|
|
return None;
|
|
}
|
|
Some(trimmed.chars().take(ERROR_BODY_PREVIEW_BYTES).collect())
|
|
}
|
|
|
|
#[cfg(test)]
|
|
mod tests {
|
|
use std::sync::Arc;
|
|
|
|
use codex_api::AuthProvider;
|
|
use http::HeaderMap;
|
|
use http::HeaderValue;
|
|
use pretty_assertions::assert_eq;
|
|
use wiremock::Mock;
|
|
use wiremock::MockServer;
|
|
use wiremock::ResponseTemplate;
|
|
use wiremock::matchers::header;
|
|
use wiremock::matchers::method;
|
|
use wiremock::matchers::path;
|
|
|
|
use super::*;
|
|
|
|
#[derive(Debug)]
|
|
struct StaticRegistryAuthProvider;
|
|
|
|
impl AuthProvider for StaticRegistryAuthProvider {
|
|
fn add_auth_headers(&self, headers: &mut HeaderMap) {
|
|
let _ = headers.insert(
|
|
http::header::AUTHORIZATION,
|
|
HeaderValue::from_static("Bearer registry-token"),
|
|
);
|
|
let _ = headers.insert(
|
|
"ChatGPT-Account-ID",
|
|
HeaderValue::from_static("workspace-123"),
|
|
);
|
|
}
|
|
}
|
|
|
|
fn static_registry_auth_provider() -> SharedAuthProvider {
|
|
Arc::new(StaticRegistryAuthProvider)
|
|
}
|
|
|
|
#[tokio::test]
|
|
async fn register_environment_posts_with_auth_provider_headers() {
|
|
let server = MockServer::start().await;
|
|
let config = RemoteEnvironmentConfig::new(
|
|
server.uri(),
|
|
"environment-requested".to_string(),
|
|
static_registry_auth_provider(),
|
|
)
|
|
.expect("config");
|
|
Mock::given(method("POST"))
|
|
.and(path("/cloud/environment/environment-requested/register"))
|
|
.and(header("authorization", "Bearer registry-token"))
|
|
.and(header("chatgpt-account-id", "workspace-123"))
|
|
.respond_with(ResponseTemplate::new(200).set_body_json(serde_json::json!({
|
|
"environment_id": "env-1",
|
|
"url": "wss://rendezvous.test/cloud-agent/default/ws/environment/env-1?role=environment&sig=abc"
|
|
})))
|
|
.mount(&server)
|
|
.await;
|
|
let client = EnvironmentRegistryClient::new(server.uri(), static_registry_auth_provider())
|
|
.expect("client");
|
|
|
|
let response = client
|
|
.register_environment(&config.environment_id)
|
|
.await
|
|
.expect("register environment");
|
|
|
|
assert_eq!(
|
|
response,
|
|
EnvironmentRegistryRegistrationResponse {
|
|
environment_id: "env-1".to_string(),
|
|
url: "wss://rendezvous.test/cloud-agent/default/ws/environment/env-1?role=environment&sig=abc".to_string(),
|
|
}
|
|
);
|
|
}
|
|
|
|
#[test]
|
|
fn debug_output_redacts_auth_provider() {
|
|
let config = RemoteEnvironmentConfig::new(
|
|
"https://registry.example".to_string(),
|
|
"env-1".to_string(),
|
|
static_registry_auth_provider(),
|
|
)
|
|
.expect("config");
|
|
|
|
let debug = format!("{config:?}");
|
|
|
|
assert!(debug.contains("<redacted>"));
|
|
assert!(!debug.contains("workspace-123"));
|
|
}
|
|
}
|