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

QC Statements are not correct #24

Closed
FaizRasool opened this issue Nov 20, 2020 · 3 comments · Fixed by #26
Closed

QC Statements are not correct #24

FaizRasool opened this issue Nov 20, 2020 · 3 comments · Fixed by #26

Comments

@FaizRasool
Copy link

FaizRasool commented Nov 20, 2020

QCStatements generated by the program is not correct:

For GB-FCA PSP_PI,PSP_AI it should be (According to OB "Issue: 2.2")

DER:306e3013060604008e4601063009060704008e4601060330570606040081982702304d302630110607040081982701020c065053505f504930110607040081982701030c065053505f41490c1b46696e616e6369616c20436f6e6475637420417574686f726974790c0647422d464341

But it is generating

DER:306c3013060604008e4601063009060704008e4601060330550606040081982702304b302430220607040081982701020c065053505f50490607040081982701030c065053505f41490c1b46696e616e6369616c20436f6e6475637420417574686f726974790c0647422d464341

I feel like all of the QC Statements are not correct. Practically speaking issue i have i am able to upload this certificate to open banking directory but unable to associate it to the statement. Could be something to do with this.

OpenSSL eIDAS PSD2 Certificate Signing Request Profiles Issue 2_2.pdf

@hatstand
Copy link
Collaborator

I think this is a difference between the versions of the profile doc. See https://github.com/creditkudos/eidas/blob/master/qcstatements/qcstatements_test.go#L24

@FaizRasool
Copy link
Author

@hatstand thanks for pointing out - any chance for updating to latest?

@hatstand
Copy link
Collaborator

I suspect the association issue you're having is an issue with the OB directory rather than this and I'm afraid I don't really have the time in the near future to figure out the changes they've made here as they're not immediately obvious from the spec.

bouk added a commit to cheddar-me/eidas that referenced this issue Mar 10, 2021
bouk added a commit to cheddar-me/eidas that referenced this issue Mar 10, 2021
hatstand pushed a commit that referenced this issue Mar 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants