Fix the dnspython check and improve wording and deps for the LDAP/AD.

Review Request #11359 — Created Jan. 6, 2021 and submitted — Latest diff uploaded

Information

Review Board
release-4.0.x

Reviewers

A recent change to get_can_enable_dns() gave it the wrong return type.
While this doesn't really impact things for us, since we don't call it
ourselves anymore, we don't want to break it. This fixes the return
type, and adds a deprecation warning.

get_can_enable_ldap() is used for both LDAP and Active Directory, but
its check only referenced LDAP, and gave legacy instructions for
installing the necessary support. The new text references both services,
and the ReviewBoard[ldap] dependency.

That dependency has also been updated to include the right version
ranges for Python 2 and 3.

Tested both functions and their results.

Tested installing ReviewBoard[ldap] on Python 2 and 3.6-3.9.

Commits

Files

    Loading...