Incorrect ldapwhoami parameter in dep8 test

Bug #1973756 reported by Andreas Hasenack
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-bonsai (Ubuntu)
Fix Released
Low
Andreas Hasenack

Bug Description

python-bonsai's dep8 test at some point calls ldapwhoami like this:

ldapwhoami -Y DIGEST-MD5 -h bonsai.test -U admin -w p@ssword
ldapsearch -h bonsai.test -b "" -s base 'objectclass=*' -x -LLL +

The -h parameter, for host, no longer exists, and the above fails[1] with:

ldapwhoami: unrecognized option -�

Since the test script is not run with `set -e`, and the exit status is not checked, this failure is silently ignored.

The -h <host> parameter should be replaced with -H ldap://<host>.

1. https://autopkgtest.ubuntu.com/results/autopkgtest-kinetic/kinetic/amd64/p/python-bonsai/20220516_170521_8990d@/log.gz

Tags: bitesize
Revision history for this message
Andreas Hasenack (ahasenack) wrote :
tags: added: update-excuse update-excuses
tags: removed: update-excuse update-excuses
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

I'm working on this package because it's blocking the cyrus-sasl2 transition (due to a real bug its dep8 tests found, see bug #1973760)

tags: added: server-todo
Revision history for this message
Andreas Hasenack (ahasenack) wrote :
Revision history for this message
Andreas Hasenack (ahasenack) wrote :
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

This was committed to debian, and when they upload it to debian, we should get it via automatic syncs. I'll then close this bug manually.

tags: removed: server-todo
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Debian merged my changes, but there is still no new upload to debian with them in it.

Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

The proposed change was released in Debian version 1.3.0+ds-4. In Kinetic, we already have 1.4.0+ds-1.

Changed in python-bonsai (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.