graphql-python/graphene-django

GraphQLView should detect batch requests

Open

#967 aberto em 21 de mai. de 2020

Ver no GitHub
 (7 comments) (1 reaction) (0 assignees)Python (4.173 stars) (754 forks)batch import
help wanted✨enhancement

Description

Is there a valid reason why the GraphQLView input is validated to either only ever accept batched requests or never accept batched requests. Is it not reasonable to accept both?

https://github.com/graphql-python/graphene-django/blob/ee120c48e16dda818cb253fdc36b7053402956b0/graphene_django/views.py#L217

Guia do colaborador

Pilha de tecnologia
pythondjangographql
Domain
backendapi
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
half day
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Basic Django knowledgeGraphQL basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80
Direção de pesquisa
The issue requests modifying the GraphQLView in graphene django/views.py to detect and accept both batched and single GraphQL requests. The relevant code is at line 217 (linked in the issue). Currently, the view validates the input to expect either a list or a single object, but not both. A first time contributor should examine the parse body or dispatch method to implement logic that checks the request body type (list or dict) and processes accordingly. The goal is to allow a client to send either format without breaking existing functionality.