Suporte à timeout nas requisições ⏰

ivanaugustobds commented 6 years ago

Objetivo: evitar erro(s) 504 (especialmente para múltiplas requisições).

Inicialmente, muito obrigado pelo projeto, tem sido extremamente útil! :D Este PR:

adiciona suporte ao uso de timeout, para (como nova explicação adicionada ao README) evitar erro(s) 504
pequenas "correções" de codestyle para mais próximo possível da PSR-2 e uma padronização como um todo
não resisti, mas aproveitei pra adicionar um pouco de interface fluente ao uso do projeto, permitindo agora fazer algo como $veiculo->setTimeout(5)->buscar('GWW-6471')->dados(); encadeado numa única linha (reconheço que deveria ter feito isso num PR à parte :/)

Demo do que isso possibilitará (conforme previsto no README.md anexo à este PR):

Múltiplos erros 504 usando múltiplos proxies e sem timeout Com múltiplos proxies, mas sem timeout

Maior parte dos 504 contornados usando proxy + timeout Com múltiplos proxies e timeout

Espero que seja útil! :wink:

chapeupreto commented 6 years ago

Olá Ivan,

Muito obrigado por sua colaboração.

Vou conferir e realizar o merge em breve.

ivanaugustobds commented 6 years ago

Em tempo; fiquei meio receoso de a documentação sobre a utilidade/utilização dessa feature não ter ficado clara.. se julgar que poderia estar melhor documentado me avise que dou uma revisada nessa PR

chapeupreto commented 6 years ago

Então é só alterar mesmo, por favor, o nome do método setTimeout para timeout, atualizar na doc e já estará pronto pro merge. A propósito, também pode remover qualquer emoji que existir na doc.

Vlws!!

ivanaugustobds commented 6 years ago

Feito.

chapeupreto commented 6 years ago

Muito obrigado!

chapeupreto / sinesp

Suporte à timeout nas requisições ⏰ #29