Hello,
At the "callback" step, when trying to get the token credentials (Server#getTokenCredentials()), i'm getting the following 400 error from QuickBooks :
oauth_problem=parameter_absent&oauth_parameters_absent=oauth_verifier
This a POST request, and QuickBooks always seems to expect to get the "oauth_verifier" argument from the query string, while the league/oauth1-client library puts it in the request body :
namespace League\OAuth1\Client\Server;
abstract class Server
{
//...
public function getTokenCredentials(TemporaryCredentials $temporaryCredentials, $temporaryIdentifier, $verifier)
{
//...
$uri = $this->urlTokenCredentials();
$bodyParameters = array('oauth_verifier' => $verifier);
$client = $this->createHttpClient();
$headers = $this->getHeaders($temporaryCredentials, 'POST', $uri, $bodyParameters);
//...
}
//...
}Am I the only one getting this error ?
It looks like an incompatibility between QuickBooks and league/oauth1-client, but I'm surprised not to see anything about this error anywhere, because it won't work at all for me. I might be doing things wrong, but I still haven't found what.
What confirms my thought is that Magento implementation, which seems to have the same specificity :
https://github.com/thephpleague/oauth1-client/blob/master/src/Client/Server/Magento.php
Maybe the QuickBooks implementation should have the same workaround ?
Here is a sum up of how I build my request :
// TemporaryCredentials $temporaryCredentials : received from the previous step
// string $oauthToken : extracted from the query string
// string $oauthVerifier : " " "
// strings $consumerKey $consumerSecret and $callbackUri speak for themselves
$clientCredentials = new ClientCredentials();
$clientCredentials->setIdentifier($consumerKey);
$clientCredentials->setSecret($consumerSecret);
$quickbooks = new Quickbooks($clientCredentials);
$quickbooks->getTokenCredentials($temporaryCredentials, $oauthToken, $oauthVerifier)
Just to give it a try, I tried to override protocolHeader() and additionalProtocolParameters() in the QuickBooks class to read this parameter and add it to the query string before the request, and it worked fine :
namespace ActiveCollab\Quickbooks;
class Quickbooks extends Server
{
//...
private $tempAddedProtocolParameters = [];
protected function protocolHeader($method, $uri, CredentialsInterface $credentials, array $bodyParameters = array())
{
$prevValue = $this->tempAddedProtocolParameters;
if (isset($bodyParameters['oauth_verifier'])) {
$this->tempAddedProtocolParameters['oauth_verifier'] = $bodyParameters['oauth_verifier'];
}
$result = parent::protocolHeader($method, $uri, $credentials, $bodyParameters);
$this->tempAddedProtocolParameters = $prevValue;
return $result;
}
protected function additionalProtocolParameters()
{
return array_merge(
parent::additionalProtocolParameters(),
$this->tempAddedProtocolParameters
);
}
}(I did it very quickly, before I found the Magento implementation, that's why it's a bit different ... and ugly)
Is it really an issue, or am I doing something wrong ?
Thanks in advance
React
Typescript
Django
Laravel
Facebook
Microsoft
Google
Alibaba
D3
Tencent