99designs / 99designs/gqlgen

return an error instead of panic in GetOperationContext

Aperta
#3,828 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Go
Stelle
10.8k
Fork
1.3k
Merge medio
2g 36m
PR unite (30g)
26

Descrizione

It's not a good idea to panic in GetOperationContext. Better to return an error here instead of panic:

https://github.com/99designs/gqlgen/blob/b6d439505845b002132d3b050ec0a3a9def97045/graphql/context_operation.go#L66

Or just return a nil if the operation context is missing

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.