Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

App is wrongfully using postback #7

Open
mirceanis opened this issue Nov 2, 2018 · 1 comment
Open

App is wrongfully using postback #7

mirceanis opened this issue Nov 2, 2018 · 1 comment
Labels
bug Something isn't working

Comments

@mirceanis
Copy link
Contributor

mirceanis commented Nov 2, 2018

When using the unified request URL (https://id.uport.me/req/<JWT>) without specifying a redirect url or callback type, the app does NOT call back with a result even if there is a callback field in the JWT.

The callback behavior is fine when using the deprecated endpoints ( me.uport.me?requestToken= )

Working example

The uPort app calls back to https://example.com with the result:

me.uport:me?requestToken=eyJ0eXAiOiJKV1QiLCJhbGciOiJFUzI1NkstUiJ9.eyJjYWxsYmFjayI6Imh0dHBzOi8vZXhhbXBsZS5jb20iLCJyZXF1ZXN0ZWQiOltdLCJhY3QiOiJub25lIiwidHlwZSI6InNoYXJlUmVxIiwiaWF0IjoxNTQxMTY2Mjg0LCJleHAiOjQ2OTQ3NjYyODQsImlzcyI6ImRpZDpldGhyOjB4Y2YwM2RkMGE4OTRlZjc5Y2I1YjYwMWE0M2M0YjI1ZTNhZTRjNjdlZCJ9.DNmXjNoTkzoYAzGWM3b0JoTUytAtd1bzdUbr9NR_1yqKZNRlDtFbu8P81KjgSBEM6_Rfqx2fu3iIPne6q3qkbgA

Broken callback example

The uPort app does not call back:

https://id.uport.me/req/eyJ0eXAiOiJKV1QiLCJhbGciOiJFUzI1NkstUiJ9.eyJjYWxsYmFjayI6Imh0dHBzOi8vZXhhbXBsZS5jb20iLCJyZXF1ZXN0ZWQiOltdLCJhY3QiOiJub25lIiwidHlwZSI6InNoYXJlUmVxIiwiaWF0IjoxNTQxMTY2Mjg0LCJleHAiOjQ2OTQ3NjYyODQsImlzcyI6ImRpZDpldGhyOjB4Y2YwM2RkMGE4OTRlZjc5Y2I1YjYwMWE0M2M0YjI1ZTNhZTRjNjdlZCJ9.DNmXjNoTkzoYAzGWM3b0JoTUytAtd1bzdUbr9NR_1yqKZNRlDtFbu8P81KjgSBEM6_Rfqx2fu3iIPne6q3qkbgA

@mirceanis mirceanis added the bug Something isn't working label Nov 2, 2018
@mirceanis
Copy link
Contributor Author

It seems the app is defaulting to postback when not explicitly stated otherwise but because postback_url is missing, it does nothing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant