Closed reneerojas closed 6 years ago
Fala @reneerojas, blz ?
Obrigado por colaborar, só vou pedir para você remover do commit as alterações na pasta dist
, foi um erro meu, mas acabei de corrigir apagando e colocando no .gitignore
.
Assim que você ajustar já faço Merger, ah e também pode remover aquela info que você adicionou no README, não é necessário.
Mais uma vez, obrigado! 👍
Uma outra coisa, você alterar o arquivo credit-card.js, na parte de criptografia (new NodeRSA
). Por favor deixe explicado aqui no PR a mudança e também crie um teste para verificar se essa função está sendo chamada com os parâmetros corretamente, assim não acontece mais esse problema.
Lembre-se de alterar o arquivo na pasta src
.
Obrigado
Tudo bom Bruno? todas as alterações que fiz foi no source, apenas executei o build antes do commit. Quanto a essa questão da criptografia, o moip realmente não reconhece o cartão caso não deixe explícito o pkcs1, com isso o pagamento funciona.
Maravilha.
Só faltou tirar aquela info do READ.me que já faço o merge. =D
Vlw
Quanto ao teste do hash, checando na lib do moip, está igual ao seu. Essa validação deles que comentei é feita pelo backend deles. O mais engraçado é que na lib deles a única diferença é o tamanho da chave 2048, mas se setar como base64 com esse tamanho, retorna que o hash é inválido. Creio que o @prem-prakash tenha lido o https://github.com/travist/jsencrypt/blob/master/src/jsencrypt.js pois na linha 29 "parameters ANY DEFINED BY algorithm OPTIONAL (NULL for PKCS #1)"
Tudo bom Bruno? Fiz umas alterações pra evitar conflitos com VeeValidate, só alterei de Validator para MoipValidator e de CreditCard para MoipCreditCard.
Claro, se isso não for quebrar seus projetos, qualquer coisa deixo direto do meu git mesmo...