Doug Goldstein
Doug Goldstein
Related to #3692 as well.
This is implemented in #3818 which I've pulled into my proposed v2.44.90 at ghcr.io/cardoe/dex:v2.44.90
@johnvan7 looks reasonable. Can you squash the commits into 1? At a minimum things won't compile until all the commits are together. The last one could be split off if...
To further this along, I've pulled this PR into build that I made at https://github.com/cardoe/dex/releases/tag/v2.44.90 You can grab the container to test with at ghcr.io/cardoe/dex:v2.44.90 and report back on this...
This is part of my v2.44.90 testing build.
I've built ghcr.io/cardoe/dex:v2.44.90 at https://github.com/cardoe/dex which is a fork with some existing PRs pulled in on top of master. Please feel free to test and report back.
I'd be a -1 on this as the Microsoft connector is oauth2 and not oidc. It's designed to work with Microsoft's not quite standard v1 endpoints while their v2 endpoints...
To further this along, I've pulled this PR into build that I made at https://github.com/cardoe/dex/releases/tag/v2.44.90 You can grab the container to test with at ghcr.io/cardoe/dex:v2.44.90 and report back on this...
@sagikazarmark This is a pretty boring one but hopefully helps end users.
Probably got a 404 on https://login.microsoftonline.com/{tenant_id}/v2.0 your error came from https://github.com/coreos/go-oidc/blob/35b8e031bcac7fed73b96b09d42e6e233a6e6562/oidc/oidc.go#L256