Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error retrieving pinned certificates #14

Closed
BabyDino opened this issue Aug 4, 2017 · 5 comments
Closed

Error retrieving pinned certificates #14

BabyDino opened this issue Aug 4, 2017 · 5 comments

Comments

@BabyDino
Copy link
Contributor

BabyDino commented Aug 4, 2017

$apiContext = ApiContext::restore(BUNQ_CONFIG);
$certificateList = CertificatePinned::listing($apiContext, BUNQ_USER_ID);

Provides an error:

Argument 1 passed to bunq\Model\BunqModel::createListFromResponseArray() must be of the type array, string given, called in vendor\bunq\sdk_php\lib\Model\BunqModel.php on line 160

@OGKevin OGKevin added the bug label Aug 4, 2017
@OGKevin
Copy link
Contributor

OGKevin commented Aug 4, 2017

Hey @BabyDino,

Thanks for reporting this. Lets see if I can reproduce and see whats going on here 👍

@BabyDino
Copy link
Contributor Author

BabyDino commented Aug 4, 2017

@OGKevin FYI, I have already one certificate pinned. But that shouldn't matter :)

@OGKevin OGKevin self-assigned this Aug 4, 2017
@OGKevin
Copy link
Contributor

OGKevin commented Aug 4, 2017

Managed to reproduce, looking for a fix. Thx for reporting @BabyDino 🤗

@OGKevin
Copy link
Contributor

OGKevin commented Aug 4, 2017

We've managed to figure out what is causing it, unfortunately there is no quick fix for this and it will be fixed in a future patch 👍

@OGKevin OGKevin added this to the Patch 0.9.1 milestone Aug 4, 2017
@dnl-blkv dnl-blkv modified the milestones: v0.10.0, Patch 0.9.2 Aug 21, 2017
@OGKevin OGKevin assigned dnl-blkv and unassigned OGKevin Aug 23, 2017
@OGKevin OGKevin modified the milestones: patch 0.10.1, v0.10.0 Aug 24, 2017
@OGKevin OGKevin modified the milestones: patch 0.10.1, v0.11.0 Sep 5, 2017
@OGKevin
Copy link
Contributor

OGKevin commented Sep 7, 2017

Closing in favour of #43

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants