Skip to content

Commit

Permalink
feat(core): release v2.0.2
Browse files Browse the repository at this point in the history
  • Loading branch information
m.vela committed Nov 5, 2019
1 parent 8a5449f commit c59fcc0
Show file tree
Hide file tree
Showing 12 changed files with 35 additions and 41 deletions.
4 changes: 4 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -217,6 +217,10 @@ to bottom:
| Response | Example: PaymentResponse
+-------------------+

### Troubleshooting

See Troubleshooting.md

### Contributing

If you have an issue or a feature request, feel free to create an issue.
Expand Down
24 changes: 24 additions & 0 deletions Troubleshooting.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
# Troubleshooting

## Connectivity

### Error: wrong signature type

Full error message: `error:1414D172:SSL routines:tls12_check_peer_sigalg:wrong signature type`

This error is thrown when there is a connectivity issue with Payvision Payment servers.
Due to compatibility reasons the servers currently only support OpenSSL security level 1.

Some newer systems (like Debian Buster) increased their default security level to level 2, which makes it impossible to connect.
See: https://wiki.debian.org/ContinuousIntegration/TriagingTips/openssl-1.1.1

As a temporary workaround, the security level on a Debian Buster system can be lowered by updating a line in the `/etc/ssl/openssl.cnf` file.
Change the line

CipherString = DEFAULT@SECLEVEL=2

to

CipherString = DEFAULT@SECLEVEL=1

Revert the change once the Payvision servers are OpenSSL level 2 compatible.
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"name": "payvision/payvision-sdk-php",
"description": "Payvision PHP SDK",
"type": "library",
"version": "2.0.1",
"version": "2.0.2",
"license": "MIT",
"require": {
"php": "^7.0.13|^7.1",
Expand Down
6 changes: 3 additions & 3 deletions src/Application/Paymentlink/Service/RequestBuilder.php
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ public static function newLink(
{
$jsonBody = Builder::toArray($input);
return new ApiRequest(
'paymentlink/links',
'paymentlinks',
'POST',
self::getParameters($jsonBody),
$jsonBody['header'] ?? [],
Expand All @@ -53,7 +53,7 @@ public static function getLinkStatus(
): ApiRequest
{
return new ApiRequest(
\str_replace('{id}', $id, 'paymentlink/links/{id}'),
\str_replace('{id}', $id, 'paymentlinks/{id}'),
'GET',
[],
[],
Expand All @@ -80,7 +80,7 @@ public static function cancelLink(
{
$jsonBody = Builder::toArray($input);
return new ApiRequest(
\str_replace('{id}', $id, 'paymentlink/links/{id}/cancel'),
\str_replace('{id}', $id, 'paymentlinks/{id}/cancel'),
'POST',
self::getParameters($jsonBody),
$jsonBody['header'] ?? [],
Expand Down
4 changes: 0 additions & 4 deletions src/DataType/Date.php
Original file line number Diff line number Diff line change
@@ -1,8 +1,4 @@
<?php
/**
* @copyright Copyright (c) 2018-2019 Payvision B.V. (https://www.payvision.com/)
* @license see LICENCE.TXT
*/

declare(strict_types=1);

Expand Down
3 changes: 0 additions & 3 deletions src/Infrastructure/ApiConnection.php
Original file line number Diff line number Diff line change
Expand Up @@ -294,9 +294,6 @@ private function buildRequestArray(Request $request, array $jsonRequest = null):
{
$returnValue = [
'query' => $request->getPathParams(),
'curl' => [
\CURLOPT_SSL_CIPHER_LIST => 'DEFAULT@SECLEVEL=1',
],
];

if ($jsonRequest !== null) {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,4 @@
<?php
/**
* @copyright Copyright (c) 2018-2019 Payvision B.V. (https://www.payvision.com/)
* @license see LICENCE.TXT
*/

declare(strict_types=1);

Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,4 @@
<?php
/**
* @copyright Copyright (c) 2018-2019 Payvision B.V. (https://www.payvision.com/)
* @license see LICENCE.TXT
*/

declare(strict_types=1);

Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,4 @@
<?php
/**
* @copyright Copyright (c) 2018-2019 Payvision B.V. (https://www.payvision.com/)
* @license see LICENCE.TXT
*/

declare(strict_types=1);

Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,4 @@
<?php
/**
* @copyright Copyright (c) 2018-2019 Payvision B.V. (https://www.payvision.com/)
* @license see LICENCE.TXT
*/

declare(strict_types=1);

Expand Down
5 changes: 3 additions & 2 deletions tests/Test/Unit/Application/Response/Fake.php
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
<?php

declare(strict_types=1);

/**
* @copyright Copyright (c) 2018-2019 Payvision B.V. (https://www.payvision.com/)
* @license see LICENCE.TXT
*/

declare(strict_types=1);

namespace Payvision\SDK\Test\Unit\Application\Response;

use Payvision\SDK\Test\Unit\Application\Response\Fake\Header;
Expand Down
12 changes: 0 additions & 12 deletions tests/Test/Unit/Infrastructure/ApiConnectionTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -113,9 +113,6 @@ public function testPostRequest()
'query' => [
'id' => '67890',
],
'curl' => [
\CURLOPT_SSL_CIPHER_LIST => 'DEFAULT@SECLEVEL=1',
],
]
);
$this->mockedResponse->method('getStatusCode')->willReturn(200);
Expand Down Expand Up @@ -160,9 +157,6 @@ public function testGetRequest()
'query' => [
'id' => '67890',
],
'curl' => [
\CURLOPT_SSL_CIPHER_LIST => 'DEFAULT@SECLEVEL=1',
],
]
);
$this->mockedResponse->method('getStatusCode')->willReturn(200);
Expand Down Expand Up @@ -292,9 +286,6 @@ public function testTolerantReader()
'query' => [
'id' => '67890',
],
'curl' => [
\CURLOPT_SSL_CIPHER_LIST => 'DEFAULT@SECLEVEL=1',
],
]
);
$this->mockedResponse->method('getStatusCode')->willReturn(200);
Expand Down Expand Up @@ -353,9 +344,6 @@ public function testExceptionOnMissingRequiredField()
'query' => [
'id' => '67890',
],
'curl' => [
\CURLOPT_SSL_CIPHER_LIST => 'DEFAULT@SECLEVEL=1',
],
]
);
$this->mockedResponse->method('getStatusCode')->willReturn(200);
Expand Down

0 comments on commit c59fcc0

Please sign in to comment.