ecomplus / app-galaxpay

E-Com Plus app to integrate Galax Pay for recurring payments
MIT License
0 stars 1 forks source link

chore(webhook): Add logs to refactor #50

Closed wisley7l closed 1 year ago

wisley7l commented 1 year ago

referente ao #45

wisley7l commented 1 year ago

Eu tinha achado q não era tantas 😬, mas voltei e tentei não alterar tanto, acionando apenas a função de buscar pelo Tid e validar o status vindo pelo webhook, o id randonico. (pelo menos essa foi as ideias de modificações iniciais)

E também algumas variavéis relacionadas ao galax que estavam em maiúsculo e não deveriam

leomp12 commented 1 year ago

Eu tinha achado q não era tantas grimacing, mas voltei e tentei não alterar tanto, acionando apenas a função de buscar pelo Tid e validar o status vindo pelo webhook, o id randonico. (pelo menos essa foi as ideias de modificações iniciais)

Na real a ideia é não validar nada a mais do que já está sendo validado. Não quero arriscar ver novamente status não sendo atualizado em produção então não queremos adicionar aqui uma possibilidade de não processar os webhooks. Na verdade essa PR nem deveria ser um fix por enquanto, só logs melhores no processamento do webhook, uma busca por enquanto adicional (mantendo as outros exatamente como estão) por tid e também batendo log.

Logs adicionais pelo type logo no início e as variáveis que jogaram pra 404 no refactor: https://github.com/ecomplus/app-galaxpay/blob/b21a19f95ce6c52b50dd213bc2b3211c7f080a76/functions/routes/galaxpay/webhooks.js#L63

Somente logs pra clarear o que deu errado...