Custom URL scheme clarification needed

Comments (12)

1. 

   Kristina Yasuda

   related to #1199 that also proposes current best option..

   • 2021-02-03T04:40:51+00:00
2. 

   Tom Jones

   I see no issue here to discuss. I have been using it for years and it works as advertised. It is designed for native apps which do not use service workers.

   The fact that the user experience is unacceptable is addressed elsewhere.

   I would close this issue.

   • 2021-02-03T17:21:11+00:00
3. 

   Kristina Yasuda

   • changed status to open

   • 2021-02-09T04:59:49+00:00
4. 

   Adam Lemmon

   I wanted to raise our concerns around requiring the openid:// scheme.

   The cases we’d like to discuss / consider are:

   1. Support for various deployment architectures such as PWAs or cloud servers likely behind https://
   2. The holder has multiple wallets on a single device
   3. The holder has multiple wallets across multiple devices

   ‌

   • 2021-02-17T00:43:01+00:00
5. 

   Tom Jones

   As i pointed out above, the scheme works as advertised. It is not needed for PWA as those are indistiguishable for IdPs,

   The wallet problem is something else and belongs in a separate thread.

   • 2021-02-17T00:50:01+00:00
6. 

   Adam Lemmon

   Hi Tom, right yes I would agree

   My comment is regarding SIOP V2 which notes:

   “Self-Issued OP MUST associate a custom schema openid:// with itself. Relying Party MUST call openid:// when sending a request to a Self-Issued OP.”

   And I am suggesting this not be a MUST and we should accommodate other schemes such as https://

   But perhaps this was the incorrect issue to raise this in then?

   • 2021-02-17T01:03:30+00:00
7. 

   Tom Jones

   RIght - that spec is cast in stone - don’t mess with it - SIOP is headed to be its own spec and can say whatever it want to say.

   • 2021-02-17T01:13:27+00:00
8. 

   Adam Lemmon

   I’d sure hope we can discuss and collaborate further before any etching into stone occurs :)

   • 2021-02-17T01:27:34+00:00
9. 

   Kristina Yasuda

   Propose to resolve this as this has been addressed in PR #25 that has been merged.

   • 2021-07-14T19:24:52+00:00
10. 

    Adam Lemmon

    Thanks @Kristina Yasuda I’d agree this seems ok to close based on the dialogue but would defer to @Oliver Terbu if his question has been answered

    • 2021-07-15T16:31:22+00:00
11. 

    Oliver Terbu

    Yes, I’m fine with closing this issue.

    • 2021-08-05T12:31:56+00:00
12. 

    Kristina Yasuda

    • changed status to resolved

    Agreed to resolve at 08-05-2021 call

    • 2021-08-06T02:09:11+00:00
13. Log in to comment