# Summary

OpenID Connect is the successor to OpenID.

# Project URL

https://www.drupal.org/project/openid_connect

# Where is the code?

https://www.drupal.org/project/openid_connect

# Estimated completion date

N/A

# Dependencies

N/A

# Who's doing the port?

sanduhrs

# What help do they need?

Reviews, Testing, 8.x Branch

# D8 roadmap

https://www.drupal.org/project/issues/openid_connect?version=8.x

# Background and reference information

Comments

sanduhrs created an issue. See original summary.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented
Status: Active » Needs review

I got a first working version.
Please review.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented

A lot more cleanups have been done.

pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins commented

Hi, I've just seen this issue, thanks for starting a port! I'll have a go at reviewing it

kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
CreditAttribution: kim.pepper at PreviousNext commented

Awesome @sanduhrs!

/me takes a look

kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
CreditAttribution: kim.pepper at PreviousNext commented

There was a fatal going to the settings form, so I fixed a couple of things.
https://github.com/sanduhrs/drupal-openid_connect/pull/2

Also added a simple web test for the settings form.

There were a number of issues with the config schema, which I ignored in this PR.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented
Title: D8 port » [openid_connect] OpenID Connect
Project: OpenID Connect / OAuth client » Drupal 8 Contrib Porting Tracker
Version: 7.x-1.x-dev » 8.x-0.x-dev
Component: Code » Module
Issue summary: View changes
Issue tags: -porting +Plan
kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
CreditAttribution: kim.pepper at PreviousNext commented

Fixed up the schema issues with the help of @benjy.

Latest version is here:
https://github.com/kimpepper/drupal-openid_connect/tree/fix-admin-form

Or you can look at the PR for sanduhrs repo. https://github.com/sanduhrs/drupal-openid_connect/pull/2

Let me know if you'd like me to post a patch here.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented

Cool, merged your changes. Thx @kim.pepper

pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins commented

@sanduhrs I've added you as a maintainer to the drupal.org project - feel free to push your 8.x-1.x branch

kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
CreditAttribution: kim.pepper at PreviousNext commented

@sandhurs you might want to do git checkout --branch 8.x-1.x --orphan when creating the branch. Not sure there is much point in keeping history as it's largely a re-write. Thoughts?

joe-b’s picture

joe-b CreditAttribution: joe-b at Wunder commented

I've created a pull request to add support for the Google 'hosted domain' authentication parameter, which limits authentications to a specific domain hosted by Google.

Pull request is at https://github.com/sanduhrs/drupal-openid_connect/pull/5

cf. https://developers.google.com/identity/protocols/OpenIDConnect#hd-param

bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
Status: Needs review » Needs work

"needs review" means there's an alpha or beta tagged. The code is not even on drupal.org yet.

(Great work, though!)

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented
Status: Needs work » Needs review

Just pushed a 8.x-1.x branch and tagged a first 8.x-1.0-alpha1.
Anyone up for a review?

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented

And can we get a development release for the issue tracker?

joe-b’s picture

joe-b CreditAttribution: joe-b at Wunder commented

I don't see a 8.x-1.x branch on the module page or issue tracker - am I missing something?

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented

see #15.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented

The 8.x-1.x branch is now visible on the project page and on the issue tracker.
A 8.x-1.0-alpha1 has been released - thanks!

pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins commented

Hi @sanduhrs yes I just did that and gave you the permission to administer releases too, sorry I'd missed that you didn't have it

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented

Thanks @pjcdawkins !

joe-b’s picture

joe-b CreditAttribution: joe-b at Wunder commented

Thanks for the great work done by all on the module D8 upgrade candidate.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented

After fixing some bugs I just released 8.x-1.0-alpha2.
Thanks for reporting!

Issue #2681221 by sanduhrs: impossible to create new user
Issue #2681293: Error: Call to a member function getFileUri() on null
Remove legacy user picture handling add support for files.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented
Issue summary: View changes
sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented
Issue summary: View changes
MustangGB’s picture

MustangGB CreditAttribution: MustangGB commented
Category: Task » Plan
sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented
Assigned: Unassigned » sanduhrs

openid_connect 8.x-1.0-beta2 release is available

tormi’s picture

tormi
Primary language Estonian
Location Tallinn
CreditAttribution: tormi as a volunteer commented
Issue summary: View changes

Deleted double content

chepec’s picture

chepec CreditAttribution: chepec commented

I'm trying to use this module for a Drupal 8 so the users can login with Google Apps, but I'm stumped since the companion module OpenID connect SSO does not have a Drupal 8 version. Does this mean that this module is a dead-end (for the time being) with Drupal 8? Or is there perhaps some other OpenID implementation I should be looking at?

pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins commented

@chepec you don't need OpenID Connect SSO to let users log in to a site with Google Apps.

OpenID Connect SSO is for creating single sign-on (cookie sharing) between a group of your own sites.

chepec’s picture

chepec CreditAttribution: chepec commented

Thanks for the clarification @pjcdawkins. I was looking for a "Login with Google" button or similar to show itself, and didn't notice the "OpenID connect" block in the list of available blocks (my only excuse is that I'm still a little new to Drupal 8).

magick93’s picture

magick93 CreditAttribution: magick93 commented

Hello

I am successfully able to login to my drupal app using the OpenID Connect module, but roles are not copied from the OpenID Connect source to Drupal.

Is this a bug, or missing feature?

mmjvb’s picture

mmjvb CreditAttribution: mmjvb as a volunteer commented
Status: Needs review » Fixed

https://www.drupal.org/project/openid_connect/releases/8.x-1.1

apaderno’s picture

apaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
CreditAttribution: apaderno as a volunteer commented

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.