Skip to content

Commit 6e56c7a

Browse files
committed
SDK-1019: Ensure serial number is cast to a string
1 parent 4c2200a commit 6e56c7a

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

src/Yoti/Util/Profile/AnchorConverter.php

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
use phpseclib\File\X509;
88
use Attrpubapi\Anchor;
99
use Yoti\Entity\Anchor as YotiAnchor;
10+
use phpseclib\Math\BigInteger;
1011

1112
class AnchorConverter
1213
{
@@ -160,7 +161,12 @@ private static function convertCertToX509($certificate)
160161
{
161162
$X509 = new X509();
162163
$X509Data = $X509->loadX509($certificate);
163-
return json_decode(json_encode($X509Data), false);
164+
$decodedX509Data = json_decode(json_encode($X509Data), false);
165+
$decodedX509Data
166+
->tbsCertificate
167+
->serialNumber
168+
->value = (string) $X509Data['tbsCertificate']['serialNumber'];
169+
return $decodedX509Data;
164170
}
165171

166172
/**

tests/Util/Profile/AnchorConverterTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ private function parseFromBase64String($anchorString)
102102
private function assertSerialNumber($anchor, $serial_number)
103103
{
104104
$cert = $anchor->getOriginServerCerts()[0];
105-
$this->assertEquals($serial_number, $cert->tbsCertificate->serialNumber->value);
105+
$this->assertSame($serial_number, $cert->tbsCertificate->serialNumber->value);
106106
}
107107

108108
/**

0 commit comments

Comments
 (0)