[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-09-29 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support pbrezina commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5283 * `master` * 0b069085cc6cb472b6c8841a26107ee1d48222ee - Add dyndns_auth_ptr support """ See the full comment at https://github.c

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-09-29 Thread joakim-tjernlund
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support joakim-tjernlund commented: """ > Do I understand it correctly that this patch fixes #5274? If yes, can you > please add: > > ``` > Resolves: https://github.com/SSSD/sssd/issues/5274 > ``` > > to the commit

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-09-29 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support pbrezina commented: """ Do I understand it correctly that this patch fixes https://github.com/SSSD/sssd/issues/5274? If yes, can you please add: ``` Resolves: https://github.com/SSSD/sssd/issues/5274? ``` to

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-09-28 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support sumit-bose commented: """ Hi, after some reading I tested this patch with the bind update policy `grant * tcp-self * PTR;` for the reverse zone. This allows a client with a matching IP address to update its

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-09-15 Thread joakim-tjernlund
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support joakim-tjernlund commented: """ > Hi, > > I'm fine with the patch but I still would like to understand how to test this > behavior. Which settings do you need for your environment > > ``` > dyndns_auth = non

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-09-15 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support sumit-bose commented: """ Hi, I'm fine with the patch but I still would like to understand how to test this behavior. Which settings do you need for your environment dyndns_auth = none dyndns_auth_pt

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-09-15 Thread joakim-tjernlund
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support joakim-tjernlund commented: """ @sumit-bose , mind setting this PR in Reviewed status? """ See the full comment at https://github.com/SSSD/sssd/pull/5283#issuecomment-692733807 ___

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-09-07 Thread joakim-tjernlund
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support joakim-tjernlund commented: """ I would really appreciate if this could be merged to master, then I can install sssd on a few more computers that uses master """ See the full comment at https://github.com/SS

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-09-01 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support alexey-tikhonov commented: """ > > Is there any reason to have two patches instead squashing into single? > > Just wanted to show what I changed, I can squash into one commit, NP Thank you. """ See the full

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-09-01 Thread joakim-tjernlund
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support joakim-tjernlund commented: """ > Hi, > > thank you for the patch. Except a minor issue I'm fine with the patch. Thanks > > I wonder if you can give an example configuration how to configure a DNS > server

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-09-01 Thread joakim-tjernlund
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support joakim-tjernlund commented: """ > Is there any reason to have two patches instead squashing into single? Just wanted to show what I changed, I can squash into one commit, NP """ See the full comment at https

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-09-01 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support alexey-tikhonov commented: """ Is there any reason to have two patches instead squashing into single? """ See the full comment at https://github.com/SSSD/sssd/pull/5283#issuecomment-685037385

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-09-01 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support sumit-bose commented: """ Hi, thank you for the patch. Except a minor issue I'm fine with the patch. I wonder if you can give an example configuration how to configure a DNS server for asymmetrical authentic

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-08-27 Thread joakim-tjernlund
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support joakim-tjernlund commented: """ @sumit-bose , could you have a look at this PR? """ See the full comment at https://github.com/SSSD/sssd/pull/5283#issuecomment-681910420 __

[SSSD] [sssd PR#5283][comment] Add dyndns_auth_ptr support

2020-08-19 Thread joakim-tjernlund
URL: https://github.com/SSSD/sssd/pull/5283 Title: #5283: Add dyndns_auth_ptr support joakim-tjernlund commented: """ See issue https://github.com/SSSD/sssd/issues/5274 """ See the full comment at https://github.com/SSSD/sssd/pull/5283#issuecomment-675923680 ___