Skip to content

Conversation

ilmax
Copy link
Contributor

@ilmax ilmax commented Sep 5, 2024

Changes

The MongoDB.Driver has been updated to the latest version to allow Akka and Aspire to work well together again. This fixes an Issue introduced by the MongoDB.Driver because latest version is now strong named.

More info can be found here, the issue this change is the same as this one.

Not sure what kind version bump this change requires on the Akka side

Checklist

For significant changes, please ensure that the following have been completed (delete if not relevant):

@ilmax
Copy link
Contributor Author

ilmax commented Sep 12, 2024

@Aaronontheweb any chance we can bump to latest MongoDB Driver? That would allow using Akka.Persistence.MongoDb with the 8.2 version of Aspire.MongoDB.Driver (and also to whoever updated the mongodb driver to latest)

@Aaronontheweb
Copy link
Member

@Aaronontheweb any chance we can bump to latest MongoDB Driver? That would allow using Akka.Persistence.MongoDb with the 8.2 version of Aspire.MongoDB.Driver (and also to whoever updated the mongodb driver to latest)

Yep, fine with me

@Aaronontheweb Aaronontheweb merged commit 8aa836f into akkadotnet:dev Sep 12, 2024
8 checks passed
@Aaronontheweb
Copy link
Member

So wait a minute, is @ilmax the new version of this driver strong-named?

@ilmax
Copy link
Contributor Author

ilmax commented Sep 12, 2024

@Aaronontheweb yes it is, that's why this Akka update is required, otherwise I could simply reference the latest mongodb.driver and be good to go.
Essentially the mongo people made a breaking change in this latest version 🤷‍♂️

@Aaronontheweb
Copy link
Member

sigh, this is probably going to create problems for other users but I guess that's inevitable unless they plan on never upgrading their Mongo drivers

@ilmax
Copy link
Contributor Author

ilmax commented Sep 12, 2024

Yeah, I wanted to write something about versioning, but I have no idea what's the right thing to do here. If the user has no other dependency that use mongo, it's all good, if there's another dependency that references the old mongo driver, this is a problem.

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