Skip to content

Use scim-server-php lib

JulienSchneider requested to merge use-scim-server-php into main

Make use of scim-server-php (https://github.com/audriga/scim-server-php)

Note: Service Provider Configuration Endpoints (https://datatracker.ietf.org/doc/html/rfc7644#section-4) are broken for now - we are working on it

We used Postman for testing, we need to migrate/merge with insomnia


A list of TODOs:

  • Undefined method issue: Could not boot scimserviceprovider: Call to undefined method Opf\\Util\\Util::setSupportedSCIMResourceTypes()
    • Was fixed a while ago
  • Issue with vendor/autoload.php inclusion in lib/AppInfo/Application.php
    • This should now be fixed with !2 (6974e3e3)
    • @blank feel free to check our fix if you can/want
  • Issue related to spaces in AAD groups name
    • Was fixed in the scim-server-php lib
    • Folder sync from AzureAD should be fully working now (except for memberships, see bellow)
    • @blank feel free to check our fix if you can/want
  • Issue with groups members not being synced from AzureAD
    • Is this still the case?
  • firebase/php-jwt issue due to clash with same dependency with a different version being used by other nc apps -> possibly use mozart as a fix
    • Should be now fixed
  • Groups with name length > 64 chars are an issue and can't be written due to some SQL constraint most probably
    • This is a NextCloud limitation
  • Unsupported PATCH requests
    • @pierre are PATCH requests supported in the current version of the app?
    • PATCH requests are not supported by the scim-server-php lib yet
    • May need to be part of a different PR (?)
  • README update, esp. the TODO list
    • This is already done
    • May need some improvement depending on all the above
  • Service Provider Configuration Endpoints are broken
    • Should be now fixed
Edited by Stanimir Bozhilov

Merge request reports

Loading