Skip to content

OIDC integration v1 #348

Closed
Closed

Description

@nicola

v1

Ldnode side

Application side

  • Signin app (to discover the webid) @deiu
  • Signup app (to create the webid) @deiu

OIDC side

  • Fix temporary signin.html page, test webid -> oidc provider discovery
  • Test user creation api endpoint (integration w oidc)
  • Test login workflow with two OIDC providers (like localhost & freecandy.space)
  • Implement the 'verify the webid' step during login process

/api/oidc/ endpoints:

  • /api/oidc/rp - relying party callback

Have a look at the scenario to understand the steps

Activity

  1. changed the title [-]OIDC integration[/-] [+]OIDC integration v1[/+] on May 13, 2016
  2. dmitrizagidulin commented on Jun 2, 2016

    @dmitrizagidulin
    Contributor

    Moving 'fix async / race condition problem' / 'implement persistent client store' item to the next version

  3. nicola commented on Jun 2, 2016

    @nicola
    ContributorAuthor

    Didn't I implement this already?

  4. dmitrizagidulin commented on Jun 2, 2016

    @dmitrizagidulin
    Contributor

    Partly. The PR isn't working, which is fine, the functionality from it is being merged into upstream libraries.

  5. nicola commented on Jun 15, 2016

    @nicola
    ContributorAuthor

    hey @dmitrizagidulin what is the state of this?

  6. dmitrizagidulin commented on Jun 15, 2016

    @dmitrizagidulin
    Contributor

    The individual items are implemented, the issue can be closed.

  7. nicola commented on Jun 16, 2016

    @nicola
    ContributorAuthor

    Perfect!
    Let's leave till they are not in master, no?Do we have any time line to bring oidc in?

  8. dmitrizagidulin commented on Jun 16, 2016

    @dmitrizagidulin
    Contributor

    Sure thing.

    Not sure if we have a timeline specifically, but I'm tracking the remaining items at this epic here: #353

  9. melvincarvalho commented on Jan 10, 2018

    @melvincarvalho
    Contributor

    @dmitrizagidulin ok to close this now?

  10. melvincarvalho commented on Jan 24, 2018

    @melvincarvalho
    Contributor

    @dmitrizagidulin gentle bump on this. Will close this issue soon if there's not an objection.

  11. dmitrizagidulin commented on Jan 24, 2018

    @dmitrizagidulin
    Contributor

    @melvincarvalho - ah, thanks! Missed the previous query somehow.

    Yes, this should be ok to close (closing it, in fact).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions