Nancy: A extensão AsString não funciona mais

Criado em 2 fev. 2017 · 9Comentários · Fonte: NancyFx/Nancy

Pré-requisitos

[x] Eu escrevi um título descritivo para o problema
[x] Eu verifiquei que estou executando a versão mais recente do Nancy
[x] Eu verifiquei se o problema existe no modo DEBUG e RELEASE
[x] Pesquisei questões abertas e fechadas para garantir que ainda não tenham sido relatadas

Descrição

Recentemente, movemos Request.Body para um Stream longe de um RequestStream. Temos uma extensão que permite obter um corpo de solicitação como string, no entanto, é um método de extensão fora de RequestStream.

Passos para reproduzir

Para fazer isso funcionar, você tem que fazer ((RequestStream)this.Body)>AsString() que é desagradável.

Isso precisa ser corrigido https://github.com/NancyFx/Nancy/blob/be4f8d42076e4e568a3742715437868e6c7d05af/src/Nancy/Extensions/RequestStreamExtensions.cs#L18

Acho que este teste precisa de uma emenda https://github.com/NancyFx/Nancy/blob/be4f8d42076e4e568a3742715437868e6c7d05af/test/Nancy.Tests/Unit/Extensions/RequestStreamExtensionsFixture.cs

Configuração do sistema

Versão de Nancy: 2.0-clinteastwood
Anfitriã Nancy
- [] ASP.NET
- [] OWIN
- [] Auto-hospedado
- [x] Outro: Kestrel
Outros pacotes e versões Nancy:
Ambiente (sistema operacional, versão e assim por diante): OSX, mas se aplicará a todos
Versão do .NET Framework:
Informações adicionais:

Up for Grabs Improvement

Fonte

jchannon

Comentários muito úteis

Não seria mais rápido enviar um PR do que escrever um problema e vincular coisas?

Sem mencionar que é sempre bom ter PRs como este para quem está começando :)

JosephWoodward em 2 fev. 2017

👍2

Todos 9 comentários

Portanto, deve ser um método de extensão em Stream vez disso, é isso que você está dizendo?

thecodejunkie em 2 fev. 2017

sim

Em 2 de fevereiro de 2017 às 09:45, Andreas Håkansson [email protected]
escreveu:

Portanto, deve ser um método de extensão no Stream, é assim que você
estão dizendo?
-
Você está recebendo isto porque é o autor do tópico.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/NancyFx/Nancy/issues/2698#issuecomment-276911614 ou mudo
o segmento
https://github.com/notifications/unsubscribe-auth/AAGapv0t5LUUld6nY8OTPMpmlkL_nGkUks5rYaWogaJpZM4L049J
.

jchannon em 2 fev. 2017

Não seria mais rápido enviar um PR do que escrever um problema e vincular coisas? 🤣

thecodejunkie em 2 fev. 2017

Provavelmente, mas ocupado e não quero esquecer :)

Em 2 de fevereiro de 2017 às 12h07, Andreas Håkansson [email protected]
escreveu:

Não seria mais rápido enviar um PR do que escrever um problema e vincular coisas?
🤣
-
Você está recebendo isto porque é o autor do tópico.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/NancyFx/Nancy/issues/2698#issuecomment-276940607 ou mudo
o segmento
https://github.com/notifications/unsubscribe-auth/AAGapgpqrQLA5OdN2U1LuVLLlMRQjjUPks5rYccNgaJpZM4L049J
.

jchannon em 2 fev. 2017

Não seria mais rápido enviar um PR do que escrever um problema e vincular coisas?

Sem mencionar que é sempre bom ter PRs como este para quem está começando :)

JosephWoodward em 2 fev. 2017

👍2

Faça assim ...

jchannon em 2 fev. 2017

Não seria mais rápido enviar um PR do que escrever um problema e vincular coisas?

Sim, é mais rápido consertar você mesmo, mas isso remove as oportunidades para iniciantes de código aberto ou mesmo veterinários experientes (que estão em busca de sugestões de valor) de se envolverem com o projeto. Confira @yourfirstpr - https://yourfirstpr.github.io/ para mais informações. Sem sangue novo; projeta atrofia.