Skip to content

Add 'referrals' to the LDAPError exception as a list containing all referrals returned from server - #459

Open
nickfolino wants to merge 1 commit into
python-ldap:mainfrom
nickfolino:referral_br
Open

nickfolino wants to merge 1 commit into
python-ldap:mainfrom
nickfolino:referral_br

Conversation

@nickfolino

Copy link
Copy Markdown

Added a refList that contains all referrals returned from the server.
Still maintains the first referral in 'info'.

@tiran tiran left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Every Python API call requires an explicit error check.

Please also include documentation update and test cases.

Comment thread Modules/constants.c Outdated
Comment thread Modules/constants.c Outdated
Comment thread Modules/constants.c Outdated
Comment thread Modules/constants.c Outdated
@mistotebe mistotebe added this to the 3.5.0 milestone Aug 28, 2026
@mistotebe mistotebe changed the title Added 'refList' as a list containing all referrals returned from server. Add 'referrals' to the LDAPError exception as a list containing all referrals returned from server Sep 10, 2026

@droideck droideck left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants