search

YunoHost-Apps / dex_ynh

Federated OpenID Connect Provider package for YunoHost
https://dexidp.io/
GNU Affero General Public License v3.0
6 stars 4 forks source link

How to set up Dex with Cloudflare Zero Trust? #22

Closed BRGustavoRibeiro closed 1 year ago

BRGustavoRibeiro commented 1 year ago

Disclaimer: I'm really not used to working with LDAP/OpenID/Dex concepts at all. I have an idea of how everything works, but nothing too complex. I'm doing my best to understand it all, so please forgive me if this is a very basic question. 🙂

Describe the bug

I'm not sure if this is a bug.

As YunoHost's admin panel don't have many security features (no captcha, no 2FA, no multiple admins), I'm trying to set up a free Cloudflare Zero Trust instance into the /yunohost/admin/ subdirectory on my domain.

This is completely possible and already works like a charm, but the cherry on top would be syncing YunoHost's LDAP with Cloudflare ZT auth. This is only possible via OpenID Connect and this wonderful module.

I was trying to set up Dex to get a cool, simple OpenID Connect I could link with Cloudflare ZT.

Here is the information that Cloudflare asks for: image

The issue here: while trying to install Dex, it asks for the "OIDC secret of the app you want to connect to the OIDC auth flow". Cloudflare does not provide such OIDC secret.

Here's how I'm trying to install Dex currently: image

I've inserted a random string in the OIDC secret input - I would expect at least to get the credentials that Cloudflare is requesting at this documentation, but nothing is returned after the installation and I honestly have no idea how to get that info, as it wasn't provided to me during the installation of this module.

What am I doing wrong? 😕

Context

Expected behavior

I expected to perform a quick install - only insert the Callback URL, and after the install finishes, it would show me a page, a popup, or anything with the data that Cloudflare was requesting.

Steps to reproduce

N/A - No errors

Logs

N/A - No errors

Limezy commented 1 year ago

Hi ! I have to admit that dex_ynh documentation is very far from optimal...

What I would try for dex installation :

What I would try for Cloudflare ZT setup :

For the certificate URL, I'm not very sure of what it is... The Outline_ynh app for which I have used Dex as the LDAP <-> OIDC connector requires me to input a "user info" URL, which I've put as https://yourserver.com/dex/userinfo Is it what they mean by "certificate URL" ? I don't know !

Limezy commented 1 year ago

OK, after checking quickly, you should be good using https://yourserver.com/dex/keys for the "Certificate URL" required value on Cloudflare ZT. Tell me if it worked !

BRGustavoRibeiro commented 1 year ago

Hey, bro! Thank you for helping me to sort this out.

I've done it correctly, but Dex didn't like Cloudflare's URL. Is this a configuration error on my side or Cloudflare's side? Is there anyway I can get a more detailed log? image

Thanks again! 🙂 _I'll consider doing some PRs to help a bit with dexynh docs after this.

Limezy commented 1 year ago

Hi ! It's with pleasure. Is the screenshot you show here still on the dex domain ? I don't know on cloudflare side, but you should be able to read some better logs of the error by going to Yunohost administration, then services, then dex and have a look there

Limezy commented 1 year ago

image

Seen on dex github. I think you should definitely check the callback url you've put. Is your team name really randomteamname ?

BRGustavoRibeiro commented 1 year ago

Everything is on Dex domain. I'm changing the domain I'm using on everything I post here (including my previous screenshot, I've used Inspect Element to change the domain) because I'm quite paranoid with security, but I can send you in private if needed. image

I should also point that I'm not running Dex from /dex, but from /advanced-res/oid. Here are my logs for /var/log/dex/dex.log.

time="2022-09-26T15:12:15Z" level=info msg="Dex Version: , Go Version: go1.17.13, Go OS/ARCH: linux amd64"
time="2022-09-26T15:12:15Z" level=info msg="config issuer: https://domain.tld/advanced-res/oid"
time="2022-09-26T15:12:15Z" level=info msg="config storage: sqlite3"
time="2022-09-26T15:12:15Z" level=info msg="config static client: openidssoauth"
time="2022-09-26T15:12:15Z" level=info msg="config connector: ldap"
time="2022-09-26T15:12:15Z" level=info msg="config refresh tokens rotation enabled: true"
time="2022-09-26T15:12:15Z" level=info msg="keys expired, rotating"
time="2022-09-26T15:12:15Z" level=info msg="keys rotated, next rotation: 2022-09-26 21:12:15.559447493 +0000 UTC"
time="2022-09-26T15:12:15Z" level=info msg="listening (http) on 127.0.0.1:5556"
time="2022-09-26T15:12:45Z" level=error msg="Failed to parse authorization request: Unregistered redirect_uri (\"https://randomteamname.cloudflareaccess.com/cdn-cgi/access/callback\")."
time="2022-09-26T15:25:47Z" level=error msg="Failed to parse authorization request: Unregistered redirect_uri (\"https://randomteamname.cloudflareaccess.com/cdn-cgi/access/callback\")."
time="2022-09-26T15:27:11Z" level=error msg="Failed to parse authorization request: Unregistered redirect_uri (\"https://randomteamname.cloudflareaccess.com/cdn-cgi/access/callback\")."
BRGustavoRibeiro commented 1 year ago

I believe I found the issue.

On Dex config.yaml, I found this: image

Let me test - I believe it will work this time 👍

BRGustavoRibeiro commented 1 year ago

Yay! The login worked after I've fixed the double HTTPS issue.

Now we have one more error. This a fresh new one 😅 image

BRGustavoRibeiro commented 1 year ago

I found this comment at Cloudflare forums. Maybe its related to /keys? image

Limezy commented 1 year ago

This seem like a last small tweak to be made, I believe in the "scopes" thing. May I let you google / search the dex documentation ?

Dex_ynh was originally designed as an ID provider for Outline_ynh, but I may need to add a few options at installation stage to cover for other cases such as yours.

I'm very interested to see whether we will manage to have a working setup or not !

Limezy commented 1 year ago

Yay! The login worked after I've fixed the double HTTPS issue.

Now we have one more error. This a fresh new one 😅

image

My fault telling you to add https:// before, sorry ! This seem like a problem, I will maybe change but I'm afraid it may break some installations. Will think of it. In any case it's very much necessary to show in the readme how that uri should be written

Limezy commented 1 year ago

I found this comment at Cloudflare forums. Maybe its related to /keys? image

OK I didn't see your message before replying above. Forget these. I have searched a little bit, and I feel that at this stage it's a problem on Cloudflare side. You should maybe ask for help on their forum, as I feel Dex is correctly configured.

BRGustavoRibeiro commented 1 year ago

Heeeello again! I have an update about this.

Unfortunately, I won't be able to set this up in the short term as I'm rushing to set some things up for this project. I will try to configure the Dex-Cloudflare integration again in a few months, but for now, I will set Cloudflare ZT with my personal Github Organization SSO.

Thanks for the help, though! I'm very excited to see the future of this project with YunoHost. 🙂

Limezy commented 1 year ago

You're welcome, and feel free to reach out anytime to finish this !