Nancy: Расширение AsString больше не работает

Созданный на 2 февр. 2017 · 9Комментарии · Источник: NancyFx/Nancy

Предпосылки

[x] Я написал описательное название проблемы.
[x] Я подтвердил, что использую последнюю версию Nancy
[x] Я проверил, существует ли проблема как в режиме DEBUG и в режиме RELEASE
[x] Я просмотрел открытые и закрытые проблемы, чтобы убедиться, что о них еще не сообщалось.

Описание

Недавно мы переместили Request.Body в Stream подальше от RequestStream. У нас есть расширение, которое позволяет вам получить тело запроса в виде строки, но это метод расширения вне RequestStream.

Действия по воспроизведению

Чтобы заставить это работать, вам нужно сделать ((RequestStream)this.Body)>AsString() что неприятно.

Это требует исправления https://github.com/NancyFx/Nancy/blob/be4f8d42076e4e568a3742715437868e6c7d05af/src/Nancy/Extensions/RequestStreamExtensions.cs#L18

Я думаю, что в этот тест нужно внести поправки https://github.com/NancyFx/Nancy/blob/be4f8d42076e4e568a3742715437868e6c7d05af/test/Nancy.Tests/Unit/Extensions/RequestStreamExtensionsFixture.cs

Конфигурация системы

Версия Нэнси: 2.0-Clinteastwood
Нэнси хозяин
- [] ASP.NET
- [] OWIN
- [] Самостоятельное размещение
- [x] Другое: пустельга
Другие пакеты и версии Nancy:
Среда (операционная система, версия и т. Д.): OSX, но будет применяться ко всем
Версия .NET Framework:
Дополнительная информация:

Up for Grabs Improvement

Источник

jchannon

Самый полезный комментарий

Разве не было бы быстрее отправить PR, чем написать проблему и связать материал?

Не говоря уже о том, что всегда приятно иметь такие пиары для новичков :)

JosephWoodward 2 февр. 2017

👍2

Все 9 Комментарий

Значит, это должен быть метод расширения на Stream , вы так говорите?

thecodejunkie 2 февр. 2017

Ага

2 февраля 2017 г. в 09:45 Андреас Хоканссон [email protected]
написал:

Так что вместо этого это должен быть метод расширения в Stream.
говорят?
-
Вы получаете это, потому что вы являетесь автором темы.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/NancyFx/Nancy/issues/2698#issuecomment-276911614 или отключить звук
нить
https://github.com/notifications/unsubscribe-auth/AAGapv0t5LUUld6nY8OTPMpmlkL_nGkUks5rYaWogaJpZM4L049J
.

jchannon 2 февр. 2017

Разве не было бы быстрее отправить PR, чем написать проблему и связать материал? 🤣

thecodejunkie 2 февр. 2017

Наверное, но занят и не хочу забывать :)

2 февраля 2017 г., в 12:07, Андреас Хоканссон [email protected]
написал:

Разве не было бы быстрее отправить PR, чем написать проблему и связать материал?
🤣
-
Вы получаете это, потому что вы являетесь автором темы.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/NancyFx/Nancy/issues/2698#issuecomment-276940607 или отключить звук
нить
https://github.com/notifications/unsubscribe-auth/AAGapgpqrQLA5OdN2U1LuVLLlMRQjjUPks5rYccNgaJpZM4L049J
.

jchannon 2 февр. 2017

Разве не было бы быстрее отправить PR, чем написать проблему и связать материал?

Не говоря уже о том, что всегда приятно иметь такие пиары для новичков :)

JosephWoodward 2 февр. 2017

👍2

Сделай это так...

jchannon 2 февр. 2017

Разве не было бы быстрее отправить PR, чем написать проблему и связать материал?

Да, это быстрее исправить самостоятельно, но это лишает новичков возможности использовать открытый исходный код или даже опытных ветеринаров (которые ищут полезные предложения), чтобы участвовать в проекте. Посетите @yourfirstpr - https://yourfirstpr.github.io/ для получения дополнительной информации. Без новой крови; атрофия проектов.