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

End-to-end test for typical login #39

Open
jfelixh opened this issue Oct 28, 2024 · 13 comments
Open

End-to-end test for typical login #39

jfelixh opened this issue Oct 28, 2024 · 13 comments
Labels
housekeeping Improvements for tools and processes

Comments

@jfelixh
Copy link
Collaborator

jfelixh commented Oct 28, 2024

No description provided.

@jfelixh jfelixh added the housekeeping Improvements for tools and processes label Oct 28, 2024
@elohwine
Copy link

elohwine commented Dec 9, 2024

Hello, when can we expect this..been failing to do end to end testing..wallet giving an error

@jfelixh
Copy link
Collaborator Author

jfelixh commented Dec 9, 2024

Unfortunately, this issue is on hold right now. A lot of mocking is needed for the wallet part. But we have active deployments that are working, so the bridge should work. If you give me some more info on your error, I may be able to help.

@elohwine
Copy link

elohwine commented Dec 9, 2024

Eroooooooorrrr

That's the error after i scan the qr code from Altme wallet,, i can't see any logs with a detailed stacktrace on the bridge's logs, peharps I'm missing some information?

@elohwine
Copy link

elohwine commented Dec 9, 2024

If you could help with a working active deployments that would be appreciated, both the bridge and the compatible wallet repos

@jfelixh
Copy link
Collaborator Author

jfelixh commented Dec 9, 2024

What is your testing setup? Are you running everything locally? Do you have a public domain or a tunnel like ngrok running?

This sounds like there is an issue with the data in the QR code or like the URL in the QR code is not reachable. You could copy the qr code image and load it into a web tool that gives you the string from the qr code.

@elohwine
Copy link

elohwine commented Dec 9, 2024

Only ssi-to-oidc-bridge is local, via ngrok..using the latest altme/latao wallets google playstore.

Let me try decoding the qr code and share the string

@elohwine
Copy link

elohwine commented Dec 9, 2024

So you recommend i run everything locally?

@jfelixh
Copy link
Collaborator Author

jfelixh commented Dec 9, 2024

I would recommend running the bridge locally, like described in the readme (https://github.com/GAIA-X4PLC-AAD/ssi-to-oidc-bridge#running-a-local-deployment). Also run the client application locally.

The wallet app should run on the phone, and the Altme app on Playstore should work.

Your problem seems to be some kind of setup/config issue. Have you followed the readme description?

What are you using as your client application? Have you examined the qr code content?

@elohwine
Copy link

elohwine commented Dec 9, 2024

Im using that setup, my client is being created by the test_client.sh.

image

Decoded to this..
openid-vc://?client_id=did:key:z6MknCGsUC4cU9V1GLCpnhwzpJd78Brc3YsYyeobTBxQXa9Y&request_uri=http%3A%2F%2Flocalhost%3A5002%2F%2Fapi%2FpresentCredential%3Flogin_id%3D22ff3a2c-1a9e-4ec3-b830-cd2f8d20819a

@elohwine
Copy link

elohwine commented Dec 9, 2024

Yeh i had not used the correct ngrok url.. i have managed to authenticate now,,, thanks for your assistance

@jfelixh
Copy link
Collaborator Author

jfelixh commented Dec 10, 2024

Great to hear it works now! Since there was a localhost URL in the QR code, I assume you had not set the environment variable for the public URL?

If you have suggestions for improving the documentation, please feel free to open a new issue.

@elohwine
Copy link

Thanks Felix.I had a usecase that may require extending the bridge to use oidc4vci to issue a credential if the registration hasn't been done on the platform. I know siop works from the wallet side, but tobenable that functionality for typical WordPress websites etc.Any pointers on the bridge?

@jfelixh
Copy link
Collaborator Author

jfelixh commented Dec 19, 2024

Issuing VCs is not in the scope of what the bridge is supporting (or supposed to support). I want to keep the complexity of the bridge somewhat down. So unfortunately no pointers for that. Although, I have written an issuance utility that may be a nice reference for you: https://github.com/jfelixh/oid4vci-wizard

But depending on your exact use case, consider if you could use VCs from an existing issuer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
housekeeping Improvements for tools and processes
Projects
None yet
Development

No branches or pull requests

2 participants