99designs / 99designs/gqlgen

Enums not validated on marshal

Aberta
#2,385 1 comentário 2 reações 0 responsáveis Ver no GitHub
Linguagem predominante
Go
Estrelas
10.8k
Forks
1.3k
Merge médio
2d 36min
PRs com merge (30d)
26

Descrição

### What happened?
Enums are validated on unmarshal, but not on marshal

### What did you expect?
I'd expect validation in both cases

### versions
- v0.17.9
- go1.19.1

I'm thinking of proposing a change to add a validate check in marshal https://github.com/99designs/gqlgen/blob/master/plugin/modelgen/models.gotpl#L99. Thoughts?

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

A issue aponta para plugin/modelgen/models.gotpl, linha 99. Comece examinando esse template para entender como os enums são marshaled. Procure a lógica de validação nos caminhos de unmarshal e compare-a. Verifique se já existem testes para validação de enums, para entender o comportamento esperado. O fix provavelmente envolve adicionar uma chamada de validação no template de marshal. Verifique executando os testes relacionados ao marshaling de enums.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
go
Domínio
backend-api-design
Tipo de issue
Bug
Dificuldade
2/5
Tempo estimado
1-3 horas
Status de atividade
Estagnada
Clareza
Claramente especificada
Facilidade para iniciantes
50/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.