syncope-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ilgrosso <>
Subject [GitHub] syncope issue #74: [SYNCOPE-1270] implementation for OpenID Connect for Admi...
Date Wed, 02 May 2018 14:11:42 GMT
Github user ilgrosso commented on the issue:
    Thanks @DimaAy the code looks almost complete, and it's also working.
    I would just make some enhancements to the Admin Console management:
    1. When adding / editing an OIDC provider, validate all values that are expected to be
URIs *before* saving
    1. Move the `Has Discovery Document` flag after `Issuer`
    1. When editing an existing ODIC provider, make all the second step's fields as read-only,
and show them all despite the Discovery Document was used during create
    Side note: I see there is no yet support for [Logout](,
which is provided at least by [Azure](,
maybe you can submit a new PR for that after the current ones gets merged.


View raw message