GeventSubscriptionServer ignore schema validation errors
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 55/100
Línea de trabajo
Comienza con gevent.py alrededor de las líneas 74–75 y reproduce una suscripción que contenga un campo no válido utilizando la configuración de GraphQL AioWS indicada. Traza cómo los errores de validación del esquema producen un ExecutionResult y verifica después que esos errores se muestran en lugar de ser reemplazados por la aserción «A subscription must return an observable».
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
- GraphQL AioWS version: 0.3.1
- Python version: Python 3.8.5
- Operating System: Debian
Description
GeventSubscriptionServer ignore any schema validation errors.
I use Altair GraphQL client. When I send subscription request with schema text errors (for example: wrong field name), this error is covered by GeventSubscriptionServer and just raise "A subscription must return an observable"
I think problem in following lines in gevent.py, line 74:75:
assert isinstance(execution_result, Observable), \
"A subscription must return an observable"
There's check only for Observable, but in case of schema error its return ExecutionResult object with errors, then assert is failed, another error is raising
What I Did
Any subscription query with any schema error
- Lenguaje dominante
- Python
- Estrellas
- 270
- Forks
- 85
- Métricas de merge de PR
- Sin PR fusionados en 30 d
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 graphql-python/graphql-ws
-
using this with signals Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
graphql-python/graphql-ws#95 · 1 reacción ·
-
graphql-ws-next Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
graphql-python/graphql-ws#90 · 4 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
graphql-python/graphql-ws#89 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 25/100
graphql-python/graphql-ws#87 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
graphql-python/graphql-ws#84 · 6 comentarios · 3 reacciones ·
Todos los issues de graphql-python/graphql-ws
Issues similares
-
essnmx good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 95/100
-
[Feature] 奇物选择添加优先级 Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
syfoud/Simulated_Scepter#174 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Giskard-AI/giskard-oss#2840 · 1 comentario ·
-
A claim comment carrying the issue number is silently declined while the workflow reports success Abiertoarea: repo bug perceived difficulty: 2
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
yeti-platform/yeti#1380 ·