search

begrossi / anp-price-collector

ANP Price Collector
12 stars 4 forks source link

Captcha #2

Open raf2003 opened 6 years ago

raf2003 commented 6 years ago

Bruno, boa tarde, parabéns pelo projeto. Eu ainda não tenho experiência em Nodejs, mas pelo que vi no código, ele não trata o captcha. Acredito que na época em que você construiu, não havia o captcha que existe hoje no site. Então o código atual não está funcionando?

begrossi commented 6 years ago

Olá, raf2003, esse foi um projeto experimental, desenvolvido para um exemplo que eu precisava fazer. O Captcha já existia sim na época, mas descobri que ele não era validado pelo sistema, ou algo assim, fácil de burlar. Quem sabe você não usa ele pra estudar Node, e nos diz se ainda funciona?

Abraços, Bruno

rrecio commented 6 years ago

Oi pessoal, lembro que rodei o código uns tempos atrás, não lembro se cheguei a fazer alguma alteração. Verei quando estiver no computador e faço um pull request se for o caso. Mas lembro de ter visto o que o Bruno falou, o código simplesmente ignora o Captcha e funciona!

Sent from my iPhone

On 23 Nov 2017, at 18:03, Bruno E. Grossi notifications@github.com wrote:

Olá, raf2003, esse foi um projeto experimental, desenvolvido para um exemplo que eu precisava fazer. O Captcha já existia sim na época, mas descobri que ele não era validado pelo sistema, ou algo assim, fácil de burlar. Quem sabe você não usa ele pra estudar Node, e nos diz se ainda funciona?

Abraços, Bruno

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

raf2003 commented 6 years ago

Bruno e Rodrigo, obrigado pela atenção. Realmente, melhor oportunidade que esta não existe, para aprender node. Rodrigo, serei muito grato quando vc puder dar este retorno. abçs

rrecio commented 6 years ago

Olhei no código que eu mexi e só tem essa alteração. Porém faz tanto tempo que eu nem lembro porque alterei, mas provavelmente foi porque essa linha estava causando um bug. Depois dessa alteração passou a funcionar… Não vou abrir o PR pois não lembro os motivos certos para explicar no commit… estou informando pois isso pode ser útil pra alguém no futuro…

[]s Rodrigo

On 24 Nov 2017, at 07:13, raf2003 notifications@github.com wrote:

Bruno e Rodrigo, obrigado pela atenção. Realmente, melhor oportunidade que esta não existe, para aprender node. Rodrigo, serei muito grato quando vc puder dar este retorno. abçs

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/begrossi/anp-price-collector/issues/2#issuecomment-346792660, or mute the thread https://github.com/notifications/unsubscribe-auth/AADN1jrUnmLvly-zLQzJ1WpsxiqqsJbtks5s5pblgaJpZM4QpAZw.