Skip to content

Commit 911bed0

Browse files
authored
Move NI and Verify to Basic (#529)
1 parent 97c41bd commit 911bed0

File tree

4 files changed

+11
-7
lines changed

4 files changed

+11
-7
lines changed

src/Insights/ClientFactory.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66

77
use Psr\Container\ContainerInterface;
88
use Vonage\Client\APIResource;
9+
use Vonage\Client\Credentials\Handler\BasicHandler;
910
use Vonage\Client\Credentials\Handler\BasicQueryHandler;
1011

1112
class ClientFactory
@@ -15,7 +16,7 @@ public function __invoke(ContainerInterface $container): Client
1516
/** @var APIResource $api */
1617
$api = $container->make(APIResource::class);
1718
$api->setIsHAL(false);
18-
$api->setAuthHandlers(new BasicQueryHandler());
19+
$api->setAuthHandlers(new BasicHandler());
1920

2021
return new Client($api);
2122
}

src/Verify/ClientFactory.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66

77
use Psr\Container\ContainerInterface;
88
use Vonage\Client\APIResource;
9+
use Vonage\Client\Credentials\Handler\BasicHandler;
910
use Vonage\Client\Credentials\Handler\TokenBodyHandler;
1011

1112
class ClientFactory
@@ -18,7 +19,7 @@ public function __invoke(ContainerInterface $container): Client
1819
->setIsHAL(false)
1920
->setBaseUri('/verify')
2021
->setErrorsOn200(true)
21-
->setAuthHandlers(new TokenBodyHandler())
22+
->setAuthHandlers(new BasicHandler())
2223
->setExceptionErrorHandler(new ExceptionErrorHandler());
2324

2425
return new Client($api);

test/Insights/ClientTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ public function setUp(): void
5252
$this->api = new APIResource();
5353
$this->api->setIsHAL(false);
5454
$this->api->setClient($this->vonageClient->reveal());
55+
$this->api->setAuthHandlers(new Client\Credentials\Handler\BasicHandler());
5556

5657
$this->insightsClient = new InsightsClient($this->api);
5758
}

test/Verify/ClientTest.php

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -51,22 +51,23 @@ public function setUp(): void
5151
->setIsHAL(false)
5252
->setBaseUri('/verify')
5353
->setErrorsOn200(true)
54-
->setAuthHandlers(new Client\Credentials\Handler\TokenBodyHandler())
54+
->setAuthHandlers(new Client\Credentials\Handler\BasicHandler())
5555
->setClient($this->vonageClient->reveal())
5656
->setExceptionErrorHandler(new ExceptionErrorHandler());
5757

5858
$this->client = new VerifyClient($api);
5959
}
6060

61-
public function testUsesCorrectAuthInBody(): void
61+
public function testUsesCorrectAuth(): void
6262
{
6363
$this->vonageClient->send(
6464
Argument::that(
6565
function (RequestInterface $request) {
66-
$this->assertRequestJsonBodyContains('api_key', 'abc', $request);
67-
$this->assertRequestJsonBodyContains('api_secret', 'def', $request);
6866
$this->assertRequestMatchesUrl('https://api.nexmo.com/verify/psd2/json', $request);
69-
67+
$this->assertEquals(
68+
'Basic ',
69+
mb_substr($request->getHeaders()['Authorization'][0], 0, 6)
70+
);
7071
return true;
7172
}
7273
)

0 commit comments

Comments
 (0)