Closed Bug 2051556 Opened 2 months ago Closed 22 days ago

Review LemonLDAP details and validate there are no implementation dependencies

Categories

(Enterprise Products :: General, task, P1)

Tracking

(Not tracked)

RESOLVED FIXED
TB PoC

People

(Reporter: bsmth, Assigned: tritchie)

References

(Blocks 1 open bug)

Details

User Story

As an enterprise developer, I would like to confirm that the LemonLDAP (OIDC) config is working as expected.
No description provided.

I did a quick test with LemonLDAP. Console expects an email field in the id token. LemonLDAP doesn't include email by default. I got it working by setting IDTokenForceClaims which forces all claims (including email) to be included in the ID token.

We may want to update console to check userinfo endpoint if the id token doesn't include email.

It's fairly easy to get the console to work with lemonldap but I am seeing errors when logging in with the client, investigating.

Assignee: nobody → tritchie

The client issue I was running into was not related to LemonLDAP, I was running a local build of the client against an older build of console which did not support POST on the policies endpoint.

LemonLDAP should work once we address the issue with the id token not having an email by default.

Created a new ticket for userinfo subrequest https://bugzilla.mozilla.org/show_bug.cgi?id=2053768

/sso/logout post was also failing https://bugzilla.mozilla.org/show_bug.cgi?id=2053804

Status: NEW → ASSIGNED

Two subtickets merged, closing this as fixed

Status: ASSIGNED → RESOLVED
Closed: 22 days ago
Resolution: --- → FIXED

I'm going to mark qe-verify- for QA to skip manual testing on this one, do you agree, Todd?

Flags: qe-verify-
Flags: needinfo?(tritchie)
Summary: Review LemonLDAP details and validate there are no implmementation dependencies → Review LemonLDAP details and validate there are no implementation dependencies

Agreed, sorry for the late reply here, I'd agreed in slack earlier

Flags: needinfo?(tritchie)
You need to log in before you can comment on or make changes to this bug.