ecomplus / app-bling-erp

E-Com Plus app to integrate Bling ERP
GNU General Public License v3.0
3 stars 4 forks source link

Enfileirar eventos de callbacks do Bling #47

Closed matheusgnreis closed 3 years ago

matheusgnreis commented 3 years ago

Algumas lojas estão perdendo sincronização com o Bling, pois está desativando pelo callback. Então segundo o que informaram no ticket, eles querem ter nossa resposta 200, independente de qualquer coisa, para saber que a notificação chegou.

leomp12 commented 3 years ago

Na verdade o enhancement é enfileirar os eventos do Bling e ao mesmo tempo tentar processar assíncrono (pra evitar delay), mas isso é meio chato. Só retornar 200 é mole, mas fica muito mais suscetível a perder eventos.

Quando há resposta de erro a intenção é que exista uma retentativa, é esperado que o webhook seja enviado de novo alguns segundos depois, pelo menos é assim que funcionam sistemas de webhooks quando bem implementados.

matheusgnreis commented 3 years ago

Eles que sugeriram enviar, informei eles que poderia ser ruim, porque não significaria que tivessem de fato uma alteração, mas eles não ligam muito pra isso, por isso sugeri igual eles falaram. Valeu.

leomp12 commented 3 years ago

Uai acho que eles só querem tirar o deles fora né haha

Em ter., 15 de jun. de 2021 23:59, Matheus Reis @.***> escreveu:

Eles que sugeriram enviar, informei eles que poderia ser ruim, porque não significaria que tivessem de fato uma alteração, mas eles não ligam muito pra isso, por isso sugeri igual eles falaram. Valeu.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ecomplus/app-bling-erp/issues/47#issuecomment-861998575, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACOZELAC64WKN4E3P4ZFVV3TTAHQBANCNFSM46YF2T3Q .

leomp12 commented 3 years ago

Não está sendo enfileirado mas já está retornando sempre 200 e tratando retentativas internamente. Parece suficiente por enquanto.