humacli logging on stderr per default

Abierto
  • ¿Algo no está bien?
#809 5 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
55/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Tranquilo
Stack tecnológico
go
Área
cli

Línea de trabajo

Comienza en humacli/humacli.go, alrededor del hook Shutdown en la línea 301, e inspecciona cómo se escribe el mensaje de detención predeterminado en os.Stderr. Decide a partir de la discusión del issue si la biblioteca debería eliminar esa salida o dirigirla a través de una interfaz de logging. Se considera terminado cuando los usuarios de logging estructurado ya no reciben un mensaje predeterminado no deseado en stderr y el apagado sigue completándose normalmente.

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

Descripción

question

Currently the humacli Shutdown hook logs on os.Stderr per default: https://github.com/danielgtaylor/huma/blob/6bfeebae55ed05e1e909041c5e3692dc168cd066/humacli/humacli.go#L301

This kind of breaks our logging setup where we use structured JSON logging. I don't really see a benefit in letting the library log this, clients can easily do this themselves:

		hooks.OnStop(func() {
			logger.Info("Stopping server")

			ctx, cancel := context.WithTimeout(context.Background(), 5*time.Second)
			defer cancel()

			if err := server.Shutdown(ctx); err != nil {
				logger.Error("Error shutting down server", slog.String("error", err.Error()))
			}
		})

Would you be open to an MR for removing the fmt.Fprintln, or replacing it with a logging interface? I'd be happy to provide a PR.

Lenguaje dominante
Go
Estrellas
4.5k
Forks
299
Merge medio
2 d 8 h
PR fusionados (30 d)
18

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 danielgtaylor/huma

Todos los issues de danielgtaylor/huma

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.