Skip to content

fix: return errors upon panics or receiving unexpected responses#433

Merged
cpuschma merged 3 commits intogo-ldap:masterfrom
cpuschma:fix_logger
May 5, 2023
Merged

fix: return errors upon panics or receiving unexpected responses#433
cpuschma merged 3 commits intogo-ldap:masterfrom
cpuschma:fix_logger

Conversation

@cpuschma
Copy link
Member

@cpuschma cpuschma commented Apr 30, 2023

This replaces the package-wide logger, used in places like processMessages and reader, and returns an error instead. For backwards compatibility, errors returned from goroutine functions are stored in Conn.err and can get retrieved through Conn.GetLastError.

cpuschma added 3 commits May 5, 2023 19:54
This replaces the package-wide logger, used in places like processMessages and reader, and returns instead an error. For backwards compatibility, errors returned from goroutine functions are stored in `Conn.err` and can get retrieved through `Conn.GetLastError`
@cpuschma cpuschma merged commit b50d289 into go-ldap:master May 5, 2023
@cpuschma cpuschma deleted the fix_logger branch May 5, 2023 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant