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.

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

Allow ConnectionRefusedError to be captured as Errno::ECONNREFUSED #368

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jwedoff
Copy link

@jwedoff jwedoff commented Jul 14, 2020

Modify exceptions to allow Net::LDAP::ConnectionRefusedError to be caught with Errno::ECONNREFUSED (allowing code to start using correct exception), while maintaining compatibility with code using Net::LDAP::ConnectionRefusedError.

Display deprecation warnings for code that attempts to catch Net::LDAP::ConnectionRefusedError, rather than any time a connection refused error is raised.

@HarlemSquirrel
Copy link
Member

@jwedoff do we still need this after merging #366 ?

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.

2 participants