Skip to content

Conversation

@sdmeisner
Copy link
Contributor

BigDecimal.from_bson was calling an undefined method. This kept the Mongo driver monitoring threads from receiving server status updates in failover scenarios. Eventually Logstash was crashing with a NoServerAvailable error after a failover and failback to the original primary.

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

@ghost
Copy link

ghost commented Dec 4, 2018

Hi @sdmeisner, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in yout Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?

@sdmeisner
Copy link
Contributor Author

Hi @sdmeisner, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in yout Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?

Sure, done

@sdmeisner sdmeisner changed the title Use latest stable mongo driver and add BIgDecimal bug fix Use latest stable mongo driver and add BigDecimal and Timestamp bug fixes Dec 5, 2018
@colinsurprenant
Copy link
Contributor

Hi @sdmeisner - Thanks for your contribution! Could you please rebase this PR once #57 is merged and we can move this forward? Probably also update the library version?

@wjjpt wjjpt mentioned this pull request Apr 24, 2019
@YakobovLior
Copy link

Hey, is there an estimation when these changes will be merged?
From my understanding the update to use mongo gem version 2.6 will give the ability to connect to mongodb with SSL (fix me if I'm wrong - #50).
Thanks in advance,
Lior

@wjjpt
Copy link

wjjpt commented Apr 29, 2019

Hi, it works with 2.7.2 too, which it is the current stable version and has support for TLS hostname verification.

@colinsurprenant
Copy link
Contributor

This PR has been moved to #59, closing here.

@colinsurprenant
Copy link
Contributor

version 3.1.6 has been published which includes the changes suggested here.

@YakobovLior
Copy link

Thank you ☺

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