Atualização da lib rmccue/requests quebra a implementação de classes do sdk

[flavioamello]

Para quem ainda usa esse módulo, ontem a lib rmccue/requests lançou uma nova release que tornou algumas implementações do OAuth deste módulo não compatíveis.

O composer.json deste sdk está como "rmccue/requests": ">=1.0" que fez com que essa nova versão fosse utilizada.

Minha solução temporária foi setar no composer.json do meu projeto a versão 1.7.0 desse rmccue/requests.

[fabioselau077]

thx

[fabioselau077]

alguma atualização oficial sobre isso?

[Reinaldo92]

Estou com o mesmo problema.

[trollfalgar]

Mesma coisa aqui!

[mx-advaud]

Rodei composer update e quebrou tudo aqui. Peguei um backup de vendor>moip e vendor>rmccue e substitui na mão mesmo por hora.

[bachilli]

A lib rmccue/requests tem uma vulnerabilidade de segurança em versões anteriores a 1.8. Recomendo atualizarem.

https://snyk.io/vuln/SNYK-PHP-RMCCUEREQUESTS-1277185

[diogenesc]

Ocorreu o mesmo problema aqui.

[viniciusbig]

Sério que a MOIP ainda não lançou correção pra isso?

[bachilli]

Olá Pessoal, Desculpem a demora no retorno, estamos reestruturando algumas áreas e processos.

Este erro causando pela versão da lib rmccue/requests, pode ser corrigido da seguinte maneira:

Acesse:

vendor/rmccue/requests/library/Requests/Auth.php, e adicione um & na declaração do ultimo método, devendo ficar:

public function Register(Request_Hooks &$hooks);

sunglasses

A sugestão é alterar o pacote da dependência e não a library do moip? Acho inviável pra grandes projetos, tendo em vista que o rmccue pode ser utilizado por outros pacotes de terceiros.

[viniciusbig]

A solução é inviável em vários níveis.

A correção deveria ser feita moip-sdk-php e não na rmccue. Não faz nem sentido.
Se sua biblioteca depende de outra, você precisa se adaptar a ela, não o contrário.

E alterar manualmente arquivos instalados via composer é um problema ainda maior.
Em um deploy automático, o sistema fica inoperável até que alguem manualmente altere o arquivo
E essa alteração será sobreescrita em um próximo deploy.

A correção já foi feita pela @milazangirolame e está disponível na PR #334

MOIP, vocês poderiam olhar a PR, por favor?

[mx-advaud]

Olá Pessoal, Desculpem a demora no retorno, estamos reestruturando algumas áreas e processos.

Este erro causando pela versão da lib rmccue/requests, pode ser corrigido da seguinte maneira:

Acesse:

vendor/rmccue/requests/library/Requests/Auth.php, e adicione um & na declaração do ultimo método, devendo ficar:

public function Register(Request_Hooks &$hooks);

😎

Isso não é uma correção, é adaptação e como foi citado anteriormente pelos colegas, realizar esse tipo de ação pode gerar diversos problemas

[viniciusbig]

Nenhum update nessa thread?

[mx-advaud]

Nenhum update nessa thread?

Pelo que vi, não. Estou monitorando para poder utilizar.
Fica a impressão de que "pularam do barco"