URL: https://github.com/SSSD/sssd/pull/5732 Author: sumit-bose Title: #5732: AD: do not override LDAP data during GC lookups - 1.16 Action: opened
PR body: """ The Global Catalog contains user and group information of the whole forest and hence any Global Catalog server can be used. Currently when a Global Catalog server is looked up the data of the LDAP server is overwritten as well. I guess the original intention was to use a single server for both services.
However since the Global Catalog server can come from any domain in the forest this might overwrite the LDAP data of a DC from the local domain with the data from a AD of a remote domain and as a result lookups for users and groups from the local domain might fail since the remote DC does not has this information available at the LDAP port. In most cases this overwrite is hidden by a following lookup to find a KDC for authentication which is searched only in the local domain again where the LDAP data is overwritten again to make sure the same DC is used for LDAP and Kerberos communication. But depending on the connection timeouts and lifetime of Kerberos tickets the KDC lookup might be skipped because new credentials are not needed and as a result the wrong LDAP data is used.
To avoid this the LDAP data is now only set if the current lookup is not a Global Catalog lookup.
Resolves: https://github.com/SSSD/sssd/issues/5351
Reviewed-by: Pavel Březina pbrezina@redhat.com (cherry picked from commit 5f3b9e1d45df77bca1b2665e67bbd73b26fafbc2) """
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5732/head:pr5732 git checkout pr5732
URL: https://github.com/SSSD/sssd/pull/5732 Title: #5732: AD: do not override LDAP data during GC lookups - 1.16
alexey-tikhonov commented: """ Thank you, ACK. """
See the full comment at https://github.com/SSSD/sssd/pull/5732#issuecomment-892930511
URL: https://github.com/SSSD/sssd/pull/5732 Title: #5732: AD: do not override LDAP data during GC lookups - 1.16
Label: +Accepted
URL: https://github.com/SSSD/sssd/pull/5732 Title: #5732: AD: do not override LDAP data during GC lookups - 1.16
Label: +Ready to push
URL: https://github.com/SSSD/sssd/pull/5732 Title: #5732: AD: do not override LDAP data during GC lookups - 1.16
pbrezina commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5732
* `sssd-1-16` * 7afd36a4c4b35d72742eec2d23bd6908e635c097 - AD: do not override LDAP data during GC lookups
"""
See the full comment at https://github.com/SSSD/sssd/pull/5732#issuecomment-895073008
URL: https://github.com/SSSD/sssd/pull/5732 Title: #5732: AD: do not override LDAP data during GC lookups - 1.16
Label: +Pushed
URL: https://github.com/SSSD/sssd/pull/5732 Title: #5732: AD: do not override LDAP data during GC lookups - 1.16
Label: -Accepted
URL: https://github.com/SSSD/sssd/pull/5732 Title: #5732: AD: do not override LDAP data during GC lookups - 1.16
Label: -Ready to push
URL: https://github.com/SSSD/sssd/pull/5732 Author: sumit-bose Title: #5732: AD: do not override LDAP data during GC lookups - 1.16 Action: closed
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5732/head:pr5732 git checkout pr5732
sssd-devel@lists.fedorahosted.org