TCPSocket specs hang if client doesn't connect
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- ruby
- Área
- networking, testing-qa
Línea de trabajo
Lee el fixture de TCPSocket en library/socket/fixtures/classes.rb alrededor de las líneas 125-129, centrándote en el apagado del servidor y en Thread#join. Ejecuta las specs de TCPSocket#initialize con un cliente que no pueda conectarse y, a continuación, verifica que cerrar el servidor antes de hacer join evita que la suite se quede bloqueada y gestiona cualquier error de doble cierre.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The TCPSocket#initialize specs will hang if the client socket does not connect to the server, since the shutdown for the server expects that it will have handled a request and be wrapping up here:
https://github.com/ruby/spec/blob/master/library/socket/fixtures/classes.rb#L125-L129
I found this while adding support for the new connect_timeout keyword; we do not support hash arguments on TCPSocket#initialize at the moment, so the socket never connects. This leaves the server waiting for an incoming connection, and the Thread#join above will hang the suite.
Perhaps we should actively try to close the server socket before we join the thread, and ignore any errors on the thread from a double-close? It would allow these specs to be a bit more robust when there are errors setting up the TCPSocket before connection.
- Lenguaje dominante
- Ruby
- Estrellas
- 622
- Forks
- 402
- Merge medio
- 14 h 51 min
- PR fusionados (30 d)
- 6
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de ruby/spec
-
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
-
Improving Marshal specs Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
ruby/spec#1350 · 3 comentarios · 3 reacciones · 1 asignado ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
palladius/rails8-app-on-gcp#145 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
rubocop/rubocop-rspec#2236 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
riscv/riscv-unified-db#2624 · 1 reacción ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100