Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

🤖 fix: a Daytona setConfig call without an API key still logs every sandbox out of ssh

Abierto Apto para principiantes
  • ¿Algo no está bien?
#7,462 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
76/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
typescript
Área
authentication, desktop

Línea de trabajo

Empieza por src/node/services/daytona/daytonaAccount.ts:91 y lee el esquema de daytona.setConfig y el comportamiento de DaytonaCredentialsStore.update descritos en el issue. Añade una prueba para setConfig({}) que compruebe que el archivo de inclusión de SSH conserva sus alias y que no se revoca ningún token almacenado en caché. El issue estará terminado cuando una clave de API omitida o sin cambios ya no active la limpieza por cambios en la configuración, o cuando el esquema exija la clave y rechace su omisión.

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

Descripción

Follow-up from the Codex review of #7100 (thread: https://github.com/coder/xum/pull/7100#discussion_r4369788514, src/node/services/daytona/daytonaAccount.ts:91).

Problem

The daytona.setConfig schema says an omitted apiKey leaves the key unchanged, and DaytonaCredentialsStore.update({}) writes nothing. DaytonaAccount.setConfig still runs the settings-change cleanup after it: it drops readiness, removes every Xum alias from the ssh include file and revokes the cached tokens. So an API caller that sends {} (for example xum api) logs every running Daytona workspace out of external editors and raw ssh until the next readiness check mints new tokens. The settings page always sends apiKey, so the app itself does not hit this.

Proposed fix

Either skip onSettingsChange when the patch leaves the key unchanged (no apiKey, or the same key), or make apiKey required in the schema and drop the "omitted = unchanged" case. Add a test that setConfig({}) keeps the include file's aliases and revokes nothing.

Why it is deferred

#7100 runs its review loop under a P0/P1-only rule; P2 findings get a tracked follow-up. No caller in the app sends a patch without apiKey.

Lenguaje dominante
TypeScript
Estrellas
2.1k
Forks
139
Merge medio
3 h 41 min
PR fusionados (30 d)
955

Preparar el entorno

  • Incluye un Dockerfile o un archivo de Docker Compose
  • Sin plantilla de pull request
  • Sin guía de contribución

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 coder/xum

Todos los issues de coder/xum

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.