When searching for users by UDM mapped properties, the accepted properties are only the users' properties of the import config, not of the /etc/ucsschool/kelvin/mapped_udm_properties.json Because of that, the search filter is incomplete and all users are returned. ....................... This can be easily fixed by adjusting the user.py (router) in kelvin. @lru_cache(maxsize=1) def accepted_udm_properties() -> Set[str]: return set(ImportUser._attributes.keys()).union( set(get_import_config().get("mapped_udm_properties", [])), set(getattr(UDM_MAPPING_CONFIG, "user", [])) )
A patch is available here: https://git.knut.univention.de/univention/ucsschool/-/merge_requests/79
Please * add the comma * merge * build the Docker image * update the 3 changelogs * create a new app in the test appcenter
Thanks for the super-quick QA! - added comma - merged - rebuild the version 1.5.3 - adapted changelog [feature/kelvin] 9f52dd14b Bug #54474: changelog [feature/kelvin] 87a3a7695 Bug #54474: accepted udm properties includes values of mapped_udm_properties.json 1.5.3: digest: sha256:45a3c6c5de25a6d3db6e040add2c5cc6f08356dbc49614d81e5117847b47740e size: 3461
OK: code change OK: changelog OK: manual test
Released with version 1.5.3 If error occurs again, please clone this bug.