coder / coder/websocket

Add ReadLimit to Option structs

Abierto
#244 9 comentarios 0 reacciones 0 asignados Ver en GitHub
enhancement
Lenguaje dominante
Go
Estrellas
5.5k
Forks
372
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Would it be possible to add a field to `AcceptOptions` and to `DialOptions` that sets the read limit on the created connection? It's not as convenient to have to call the `SetReadLimit()`. If the field is 0 then a default limit can be applied.

Is there a need to change the limit dynamically? I mean, why is it a method (i.e. can be called multiple times) vs a field in those option setting structs?

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Comienza localizando las definiciones de AcceptOptions y DialOptions y el método SetReadLimit; después, sigue cómo cada opción crea una conexión. Determina cómo debe interactuar un campo con valor cero con el límite predeterminado y si el método debe seguir siendo utilizable para cambios dinámicos. La tarea estará completada cuando ambas estructuras de opciones puedan configurar el límite de lectura inicial sin eliminar el comportamiento existente del método.

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

Evaluación

Stack tecnológico
go
Área
backend-api-design, networking
Tipo de issue
Nueva funcionalidad
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
38/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.