Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bad error catching in scripts/multi_ldap_change.php #970

Closed
coudot opened this issue Sep 4, 2024 · 0 comments · Fixed by #971
Closed

Bad error catching in scripts/multi_ldap_change.php #970

coudot opened this issue Sep 4, 2024 · 0 comments · Fixed by #971
Assignees
@coudot
Labels
bug sponsor Developpement sponsorized by customers
Milestone
1.7.0

Comments

@coudot
Copy link
Member

coudot commented Sep 4, 2024

Same issue than #698 but in scripts/multi_ldap_change.php:

PHP Fatal error:  Uncaught TypeError: ldap_get_dn(): Argument #2 ($entry) must be of type LDAP\ResultEntry, bool given in /usr/share/self-service-password/scripts/multi_ldap_change.php:79
Stack trace:
#0 /usr/share/self-service-password/scripts/multi_ldap_change.php(79): ldap_get_dn()
#1 {main}
  thrown in /usr/share/self-service-password/scripts/multi_ldap_change.php on line 79
@coudot coudot added the bug label Sep 4, 2024
@coudot coudot added this to the 1.7.0 milestone Sep 4, 2024
@coudot coudot self-assigned this Sep 4, 2024
@coudot coudot mentioned this issue Sep 4, 2024
Merged
@coudot coudot linked a pull request Sep 4, 2024 that will close this issue
Only call ldap_get_dn on an entry #971
Merged
@coudot coudot added the sponsor Developpement sponsorized by customers label Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@coudot coudot

Labels
bug sponsor Developpement sponsorized by customers
Projects
None yet
Milestone
1.7.0
Development

Successfully merging a pull request may close this issue.

Only call ldap_get_dn on an entry ltb-project/self-service-password
1 participant
@coudot