nextcloud / user_saml

:lock: App for authenticating Nextcloud users using SAML https://apps.nextcloud.com/apps/user_saml
https://portal.nextcloud.com/article/configuring-single-sign-on-10.html
GNU Affero General Public License v3.0
96 stars 76 forks source link

Handle incomplete saml configuration more gracefully #840

Open kesselb opened 6 months ago

kesselb commented 6 months ago

Steps to reproduce

  1. Enable user_saml
  2. Use built-in SAML authentication
  3. Look around, do not configure saml properly
  4. Go to sleep
  5. Visit Nextcloud (the session expired, and I was redirected to the login page)
  6. Greeted by an error message

Screenshot from 2024-05-09 15-41-47

Expected behaviour

I think it would be possible to handle such an incomplete but enabled saml configuration more gracefully.

Actual behaviour

An incomplete saml configuration leads to an error message on the login page.

Incomplete saml configuration => Layer 8 problem, although we could handle it better.

Server configuration

PHP version: 8.3

Nextcloud version: master/30 (see Nextcloud admin page)

Where did you install Nextcloud from: git