Fix: don't send pg output directly to the client #16
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Vladyslav Karpenko klarpen@gmail.com
Начиная с примера
b-transportоба варианта транспорта (http & ws) возвращают результат выполнения API метода в исходном виде. Ранее именно на этом шаге выполнялась выборка только свойстваresult.rows, т.к. все API методы были завязаны на результат из БД.Поскольку результат возвращаемый методами API сервисов не изменился согласно правки транспорта — на клиента приходит оригинальный полный ответ от
pgдрайвера.Данная правка корректирует API сервисы
user&country, чтобы они возвращали масив результатов согласно контракту предыдущих примеров.Для испытания в файлы
static/client.jsдобавлял сценарийВ исходниках там только запрос к
api.talks.sayкоторый не проявляет ситуацию.Примечание №1
Решение предполагает, что ранее на мастер ветку уже был применён пул запрос #15 . В коде уже содержится соответствующая правка для
b-transportиc-commonjs, но НЕd-message. Поэтому логично применять #15 первым.Примечание №2
Правка не влючает в себя изменение кода API сервиса
city, т.к. это потребовало б его расширить аналогично другим, но он вообще не используется клиентом. Возможно целесообразно из примеровb-transportиc-commonjsвообще удалить файл./api/city.js.