Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIx: pg driver's output goes beyond database layer #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KLarpen
Copy link

@KLarpen KLarpen commented Jan 17, 2023

Signed-off-by: Vladyslav Karpenko [email protected]

Начиная с примера b-transport оба варианта транспорта (http & ws) возвращают результат выполнения API метода в исходном виде. Ранее именно на этом шаге выполнялась выборка только свойства result.rows, т.к. все API методы были завязаны на результат из БД.

Поскольку результат возвращаемый методами API сервисов не изменился согласно правки транспорта — на клиента приходит оригинальный полный ответ от pg драйвера.

Знімок екрана 2023-01-15 о 11 08 16

Данная правка корректирует скрипт db.js, чтобы все методы возвращали масив результатов, а не полный объект ответа драйвера БД. В результате ответ API методов возвращается к состоянию согласно контракту предыдущих примеров.

Знімок екрана 2023-01-15 о 12 22 00

Для испытания в файлы static/client.js добавлял сценарий

await api.country.read(3);
await api.user.update(4, { login: 'iskandar', password: 'zulqarnayn' });

В исходниках там только запрос к api.talks.say который не проявляет ситуацию.

Примечание

Решение предполагает, что ранее на мастер ветку уже был применён пул запрос #15 . В коде уже содержится соответствующая правка для b-transport и c-commonjs, но НЕ d-message. Поэтому логично применять #15 первым.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant