[SSSD] [sssd PR#690][comment] DYNDNS: Convert dyndns timer to be_ptask

2018-11-06 Thread thalman
URL: https://github.com/SSSD/sssd/pull/690 Title: #690: DYNDNS: Convert dyndns timer to be_ptask thalman commented: """ This limit was not in code, only in documentation (sssd-ad.5.xml). It is also inconsistent with sssd-ipa where there is no such limit and also default value for dyndns_refres

[SSSD] [sssd PR#690][comment] DYNDNS: Convert dyndns timer to be_ptask

2018-11-06 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/690 Title: #690: DYNDNS: Convert dyndns timer to be_ptask pbrezina commented: """ Where did you get this 60 seconds limit? I can not find it in original code: ```c ad_dyndns_init: if (period < 60) period = 60; ``` Also please remove `be_nsupdate_init_timer`

[SSSD] [sssd PR#686][comment] nss: use enumeration context as talloc parent for cache req result

2018-11-06 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/686 Title: #686: nss: use enumeration context as talloc parent for cache req result pbrezina commented: """ `nctx->pwent` and similar were not talloc pointers so sssd crashed. I pushed new patches. """ See the full comment at https://github.com/SSSD/sss

[SSSD] [sssd PR#686][synchronized] nss: use enumeration context as talloc parent for cache req result

2018-11-06 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/686 Author: pbrezina Title: #686: nss: use enumeration context as talloc parent for cache req result Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/686/head:pr686 git checkout pr