[sssd PR#817][opened] LDAP: Return the error message from the extdom password change also on failure
URL: https://github.com/SSSD/sssd/pull/817 Author: jhrozek Title: #817: LDAP: Return the error message from the extdom password change also on failure Action: opened
PR body: """ Resolves: https://pagure.io/SSSD/sssd/issue/4015
If password change fails, the tevent request would call TEVENT_REQ_RETURN_ON_ERROR before returning the error message that comes from the server, so the server message would not be propagated to the caller.
This regressed in cf1d7ff """
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/817/head:pr817 git checkout pr817
URL: https://github.com/SSSD/sssd/pull/817 Title: #817: LDAP: Return the error message from the extdom password change also on failure
jhrozek commented: """ CC @pbrezina """
See the full comment at https://github.com/SSSD/sssd/pull/817#issuecomment-495357627
URL: https://github.com/SSSD/sssd/pull/817 Title: #817: LDAP: Return the error message from the extdom password change also on failure
sumit-bose commented: """ Hi,
I think you wanted to say 'exop' instead of 'extdom' in the commit message.
bye, Sumit """
See the full comment at https://github.com/SSSD/sssd/pull/817#issuecomment-495483387
URL: https://github.com/SSSD/sssd/pull/817 Title: #817: LDAP: Return the error message from the extdom password change also on failure
pbrezina commented: """ Ack. """
See the full comment at https://github.com/SSSD/sssd/pull/817#issuecomment-495497013
URL: https://github.com/SSSD/sssd/pull/817 Title: #817: LDAP: Return the error message from the extdom password change also on failure
Label: +Accepted
URL: https://github.com/SSSD/sssd/pull/817 Author: jhrozek Title: #817: LDAP: Return the error message from the extdom password change also on failure Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/817/head:pr817 git checkout pr817
URL: https://github.com/SSSD/sssd/pull/817 Title: #817: LDAP: Return the error message from the extdom password change also on failure
jhrozek commented: """ thank you for the review, I fixed the commit message """
See the full comment at https://github.com/SSSD/sssd/pull/817#issuecomment-495513658
URL: https://github.com/SSSD/sssd/pull/817 Author: jhrozek Title: #817: LDAP: Return the error message from the extdom password change also on failure Action: closed
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/817/head:pr817 git checkout pr817
URL: https://github.com/SSSD/sssd/pull/817 Title: #817: LDAP: Return the error message from the extdom password change also on failure
jhrozek commented: """ * master: 9a4d5f0 * sssd-1-16: 4ab1b75
"""
See the full comment at https://github.com/SSSD/sssd/pull/817#issuecomment-496662020
URL: https://github.com/SSSD/sssd/pull/817 Title: #817: LDAP: Return the error message from the extdom password change also on failure
Label: +Pushed
sssd-devel@lists.fedorahosted.org
-
jhrozek -
pbrezina
-
sumit-bose