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

Fix SNI events #513

Merged
merged 5 commits into from
Oct 21, 2021
Merged

Fix SNI events #513

merged 5 commits into from
Oct 21, 2021

Conversation

mpenick
Copy link
Contributor

@mpenick mpenick commented Aug 10, 2021

When using SNI routing the driver is not mapping event address to the representation used by the driver (an Address with an IP and SNI name). It needs to be mapped using the rpc_address from the system tables.

The address in status/topology events needs to be mapped to internal
SNI address so that nodes will be properly added and removed.
@benstadin
Copy link

Just as an additional input, this branch works for us when using AWS Keyspace while we have intermittent issues with the current release. I hope this gets accepted soon.

@mpenick mpenick merged commit 07f8ade into master Oct 21, 2021
fsaporito pushed a commit to cloudian/cpp-driver that referenced this pull request May 9, 2023
* Fix events when using SNI routing

The address in status/topology events needs to be mapped to internal
SNI address so that nodes will be properly added and removed.

* Additional fixes

* Formatting fix

* Simplify logic

Co-authored-by: Bret McGuire <bret.mcguire@datastax.com>
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