Skip to content

Upgrade rxjava and fix connection handler startup wait#103

Merged
pavelhoral merged 2 commits intoWrenSecurity:mainfrom
pavelhoral:upgrade-rxjava
Jan 15, 2026
Merged

Upgrade rxjava and fix connection handler startup wait#103
pavelhoral merged 2 commits intoWrenSecurity:mainfrom
pavelhoral:upgrade-rxjava

Conversation

@pavelhoral
Copy link
Member

This PR upgrades RxJava to the latest compatible version (next PR should upgrade to the latest stable) and fixes bug in LDAPConnectionHandler2 that caused WrenSecurity/wrenam#271.

Copy link
Member

@fyrbach fyrbach left a comment

Choose a reason for hiding this comment

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

I suggest updating the copyright header.

…ization

This fixes an obvious bug of not using the waitListen property in the same  way
as in the legacy LDAPConnectionListener. Not waiting for the listener startup
caused WrenSecurity/wrenam#271.
Copy link
Member

@fyrbach fyrbach left a comment

Choose a reason for hiding this comment

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

LGTM

@pavelhoral pavelhoral merged commit 580b419 into WrenSecurity:main Jan 15, 2026
4 of 8 checks passed
@pavelhoral pavelhoral deleted the upgrade-rxjava branch February 4, 2026 07:47
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