Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Corrige Recoverable Error e Soap Error #163

Merged

Conversation

@rafaelpatro
Copy link
Contributor

@rafaelpatro rafaelpatro commented Jun 21, 2016

closes #162

  1. Corrige o tratamento do erro Soap com os Correios:

    • Could not connect to host
    • Error Fetching http headers
    • Gateway Timeout
    • SOAP-ERROR: Parsing WSDL
    • SOAP-ERROR: Parsing Schema

    Crítico: Alguns desses erros podem encaminhar e-mails para os destinos errados.
    Após a correção, o erro pode aparecer novamente no log, mas estará sendo tratado, o que não ocorria antes.

  2. Corrige o Recoverable Error: Erro de tipagem. Após a correção o erro não aparece mais no log.

Como já deixei claro em outras PRs, tenho pouco tempo pra colaborar. Se puderem testar e dar um feedback, para o @pedro-teixeira não ficar no escuro...

@rafaelpatro rafaelpatro changed the title Fix soap and recoverable Corrigige Recoverable Error e Soap Error Jun 21, 2016
@rafaelpatro rafaelpatro changed the title Corrigige Recoverable Error e Soap Error Corrige Recoverable Error e Soap Error Jun 21, 2016
@fredbrunken
Copy link

@fredbrunken fredbrunken commented Jun 21, 2016

@jarjar123 Fiz as alterações que vc sugeriu e parece que está tudo funcionando. Log está limpo agora e tudo continua funcionando.

Obrigado

@pedro-teixeira
Copy link
Owner

@pedro-teixeira pedro-teixeira commented Jun 21, 2016

@rafaelpatro esse fix merece outra release? Ou não é urgente?

@rafaelpatro
Copy link
Contributor Author

@rafaelpatro rafaelpatro commented Jun 22, 2016

Bom dia @pedro-teixeira.
Merece sim! O item 1 é bem urgente. Sinto por não ter percebido antes.
Na verdade eu deveria ter colocado uma opção de ativar/desativar o monitoramento das entregas.
Vou fazer isso o mais rápido possível.

Enquanto isso, caso alguém queira desativar o monitoramento das entregas, basta remover a tag correios_status_check do arquivo app/code/community/PedroTeixeira/Correios/etc/config.xml:

            <correios_status_check>
                <schedule>
                    <cron_expr>*/15 * * * *</cron_expr>
                </schedule>
                <run>
                    <model>pedroteixeira_correios/observer::sroTrackingJob</model>
                </run>
            </correios_status_check>
@pedro-teixeira
Copy link
Owner

@pedro-teixeira pedro-teixeira commented Jun 22, 2016

@rafaelpatro vai enviar outra PR ou espero você enviar nessa?

@rafaelpatro
Copy link
Contributor Author

@rafaelpatro rafaelpatro commented Jun 22, 2016

Farei outra PR

@pedro-teixeira pedro-teixeira added this to the v4.7.2 milestone Jun 22, 2016
@pedro-teixeira pedro-teixeira self-assigned this Jun 22, 2016
@pedro-teixeira pedro-teixeira merged commit 6c1b1e3 into pedro-teixeira:master Jun 22, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
rafaelpatro added a commit to rafaelpatro/correios that referenced this pull request Jun 22, 2016
@rafaelpatro rafaelpatro deleted the rafaelpatro:fix_soap_and_recoverable branch Jun 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants