RicoSuter/NSwag

required field on response header spec not parsed correctly

Aberta

#2.928 aberto em 29 de jun. de 2020

Ver no GitHubGuia do colaborador
 (1 comentário) (1 reação) (0 responsável)C# (1.356 forks)batch import
help wantedproject: NSwag.Core.Yamltype: bug

Métricas do repositório

Stars
 (7.358 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

The following response header YAML definition gives an error while generating C# Core 3.1 code:

  responses:
    '200':
      description: OK
      headers:
        Last-Modified:
          description: The UTC datetime at when the item was last modified
          required: true
          schema:
            type: string

Exception: Newtonsoft.Json.JsonSerializationException: Error converting value "true" to type 'System.Collections.Generic.ICollection1[System.String]'. Path 'paths['/item/{id}'].head.responses.200.headers.Last-Modified.required', line 1, position 2518. ---> System.ArgumentException: Could not cast or convert from System.String to System.Collections.Generic.ICollection1[System.String].

It looks like the generator is expecting a string array, while the required parameter should be parsed as a boolean, just like path parameters

(version 13.6.2)

Guia do colaborador

Direção de pesquisa
Investigue como o campo 'required' do cabeçalho de resposta é analisado na base de código NSwag; compare com a análise de parâmetros de caminho para identificar a coerção de tipo incorreta.
Pilha de tecnologia
csharp
Domain
backendapi
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#.NET
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60