Nancy: La extensión AsString ya no funciona

Creado en 2 feb. 2017 · 9Comentarios · Fuente: NancyFx/Nancy

Prerrequisitos

[x] He escrito un título de problema descriptivo.
[x] He verificado que estoy ejecutando la última versión de Nancy.
[x] He verificado si el problema existe en los modos DEBUG y RELEASE
[x] He buscado problemas abiertos y cerrados para asegurarme de que aún no se hayan informado

Descripción

Recientemente movimos Request.Body a Stream lejos de RequestStream. Tenemos una extensión que le permite obtener un cuerpo de solicitud como cadena, sin embargo, es un método de extensión fuera de RequestStream.

Pasos para reproducir

Para que esto funcione tienes que hacer ((RequestStream)this.Body)>AsString() que es desagradable.

Esto debe solucionarse https://github.com/NancyFx/Nancy/blob/be4f8d42076e4e568a3742715437868e6c7d05af/src/Nancy/Extensions/RequestStreamExtensions.cs#L18

Creo que esta prueba necesita una corrección https://github.com/NancyFx/Nancy/blob/be4f8d42076e4e568a3742715437868e6c7d05af/test/Nancy.Tests/Unit/Extensions/RequestStreamExtensionsFixture.cs

Configuración del sistema

Versión de Nancy: 2.0-clinteastwood
Nancy anfitrión
- [] ASP.NET
- [] OWIN
- [] Autohospedado
- [x] Otro: cernícalo
Otros paquetes y versiones de Nancy:
Entorno (sistema operativo, versión, etc.): OSX pero se aplicará a todos
Versión de .NET Framework:
Información adicional:

Up for Grabs Improvement

Fuente

jchannon

Comentario más útil

¿No sería más rápido enviar un PR que escribir un problema y vincular cosas?

Sin mencionar que siempre es bueno tener relaciones públicas como esta en juego para los principiantes :)

JosephWoodward en 2 feb. 2017

👍2

Todos 9 comentarios

Entonces, debería ser un método de extensión en Stream lugar, ¿es así como lo estás diciendo?

thecodejunkie en 2 feb. 2017

sí

El 2 de febrero de 2017 a las 09:45, Andreas Håkansson [email protected]
escribió:

Por lo tanto, debería ser un método de extensión en Stream.
¿están diciendo?
-
Estás recibiendo esto porque eres el autor del hilo.
Responda a este correo electrónico directamente, véalo en GitHub
https://github.com/NancyFx/Nancy/issues/2698#issuecomment-276911614 , o silenciar
la amenaza
https://github.com/notifications/unsubscribe-auth/AAGapv0t5LUUld6nY8OTPMpmlkL_nGkUks5rYaWogaJpZM4L049J
.

jchannon en 2 feb. 2017

¿No sería más rápido enviar un PR que escribir un problema y vincular cosas? 🤣

thecodejunkie en 2 feb. 2017

Probablemente pero ocupado y no quiero olvidar :)

El 2 de febrero de 2017 a las 12:07, Andreas Håkansson [email protected]
escribió:

¿No sería más rápido enviar un PR que escribir un problema y vincular cosas?
🤣
-
Estás recibiendo esto porque eres el autor del hilo.
Responda a este correo electrónico directamente, véalo en GitHub
https://github.com/NancyFx/Nancy/issues/2698#issuecomment-276940607 , o silenciar
la amenaza
https://github.com/notifications/unsubscribe-auth/AAGapgpqrQLA5OdN2U1LuVLLlMRQjjUPks5rYccNgaJpZM4L049J
.

jchannon en 2 feb. 2017

¿No sería más rápido enviar un PR que escribir un problema y vincular cosas?

Sin mencionar que siempre es bueno tener relaciones públicas como esta en juego para los principiantes :)

JosephWoodward en 2 feb. 2017

👍2

Hazlo así ...

jchannon en 2 feb. 2017

¿No sería más rápido enviar un PR que escribir un problema y vincular cosas?

Sí, es más rápido arreglarlo usted mismo, pero eso elimina las oportunidades para que los novatos de código abierto o incluso los veteranos experimentados (que buscan sugerencias de valor) se involucren en el proyecto. Consulte @yourfirstpr - https://yourfirstpr.github.io/ para obtener más información. Sin sangre nueva; proyectos atrofia.