search

pagarme / pagarme-core-api-php

Other
0 stars 2 forks source link

Authorization has been denied for this request. #48

Closed peuweb closed 1 year ago

peuweb commented 1 year ago

Verifiquei somente agora que a assinatura do método foi atualizada.

antes:

new PagarmeCoreApiClient($basicAuthUserName = null, $basicAuthPassword = null);

depois:

new PagarmeCoreApiClient($serviceRefererName = null, $basicAuthUserName = null, $basicAuthPassword = null);

Só não ficou muito claro que tipo de padrão devemos informar na variável $serviceRefererName.

Acredito ser um controle de qual aplicação as chamadas estão sendo realizadas já que esse valor também é passado no header do request da API.

Por hora estou passando o nome da minha aplicação em formato slug. Podemos considerar esse valor?

Exemplo:

new PagarmeCoreApiClient('my-php-application', 'my-secret-key');
prhost commented 1 year ago

Adicionam um paramentro opcional como primeira posição do método quebrando compatibilidade e lançam em uma versão de patch 5.7.X?

Semantic version chora no chuveiro.

@magnomoreira @samuelleitemundipagg

Commit: https://github.com/pagarme/pagarme-core-api-php/commit/abad0173984ed14c9b9ad18207396e16482e4513

peuweb commented 1 year ago

@prhost os caras voltaram como era antes.

Aí não tem como confiar que o versionamento é feito a nível de patch. ~5.8 esquece...

magnomoreira commented 1 year ago

Bom dia @peuweb , tudo bem ? O problema foi resolvido, retiramos o parâmetro da primeira posição que estava ocasionando o 401! correção feita na versão ~5.8.5! Desde já peço desculpas pelo o inconveniente !