Bug 56494 - 91univention-saml.inst fails if saml/idpconfig entry does not exist
91univention-saml.inst fails if saml/idpconfig entry does not exist
Status: NEW
Product: UCS
Classification: Unclassified
Component: Join (univention-join)
UCS 5.0
Other Linux
: P5 normal (vote)
: ---
Assigned To: UCS maintainers
UCS maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2023-08-28 16:52 CEST by Christina Scheinig
Modified: 2023-08-28 16:52 CEST (History)
0 users

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 4: Minor Usability: Impairs usability in secondary scenarios
Who will be affected by this bug?: 1: Will affect a very few installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.046
Enterprise Customer affected?: Yes
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number: 2023082821000191
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Christina Scheinig univentionstaff 2023-08-28 16:52:52 CEST
In the joinscript we assume the entry in saml/idpconfig already exist, if we make a modify to an entry, but creating it later.


370 if joinscript_check_version_in_range_executed "1" "9"; then
371         udm saml/idpconfig modify "$@" \
372                 --dn="id=default-saml-idp,cn=univention,$ldap_base" \
373                 --append LdapGetAttributes="memberOf" || die
374 fi
[...]
439 udm saml/idpconfig create "$@" \
440         --ignore_exists \
441         --position="cn=univention,$ldap_base" \
442         --set id=default-saml-idp || die "Error creating object. Make sure the univention-saml package is up to date on the Primary Directory Node" # Remove whe    n fixing Bug #47402

This resuls in 
++ printf '[%(%F %T)T.%06.0f]  DEBUG at %s:%s:%s: %s\n' 1693222542 312553 /usr/lib/univention-install/91univention-saml.inst 371 main 'udm saml/idpconfig modify "$@
" --dn="id=default-saml-idp,cn=univention,$ldap_base" --append LdapGetAttributes="memberOf"'
+ udm saml/idpconfig modify --dn=id=default-saml-idp,cn=univention,dc=schein,dc=ig --append LdapGetAttributes=memberOf
E: object not found