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

Migrate to .NET Core, 7.2, and NSB.Extensions.Hosting #323

Closed
jbogard opened this issue Nov 18, 2019 · 8 comments · Fixed by #348
Closed

Migrate to .NET Core, 7.2, and NSB.Extensions.Hosting #323

jbogard opened this issue Nov 18, 2019 · 8 comments · Fixed by #348

Comments

@jbogard
Copy link
Contributor

jbogard commented Nov 18, 2019

About time for an upgrade I think :)

net461 -> netcoreapp3.0
ASP.NET Core 2.x -> ASP.NET Core 3.0
MSMQ -> ??? (RabbitMQ?)

@jbogard
Copy link
Contributor Author

jbogard commented Nov 18, 2019

Also, remove Windsor as the container, I don't think it's needed at all any more.
Remove OWIN hosting, use the ApiController instead.

@dvdstelt
Copy link
Member

It's unlikely we will pick up the suggested changes any time soon, if at all. The workshop currently fits our purpose and the suggested changes wouldn't have an impact on the theories we'd like to share with attendees.
Besides that, with the very rare occasional exception, you are the only one left presenting this workshop at conferences.

But you're welcome to submit a PR, though. :-)
I would also suggest moving to LearningTransport, as this requires less set-up from attendees. With the PlatformSample package, the Particular Service Platform can be used as well.

@dvdstelt
Copy link
Member

@jbogard I just updated it to Learning Transport. That should make it more easy for people to complete the exercises without having MSMQ installed.

No idea if it now works in Visual Studio for Mac or anything. If I can, I'll have someone try that. It still needs SQL for persisting Entity Framework stuff.

@jbogard
Copy link
Contributor Author

jbogard commented Jan 27, 2020 via email

@jbogard
Copy link
Contributor Author

jbogard commented Feb 10, 2022

Just an FYI, I have a version of this that is all on .NET 6:

https://github.com/jbogard/presentations/tree/master/DistributedTracing/Example

@dvdstelt
Copy link
Member

Interesting! I'll have a look. Thanks for the ping!

@dvdstelt
Copy link
Member

@jbogard I used what you created here and removed OpenTelemetry stuff. Modified some stuff and then migrated to LiteDb. Just a NuGet package that should work with multiple resources accessing the database, so it doesn't lock up. It also has a LiteDb Studio with which you can have a look at the database. Instructions have been updated as well. Should not be too hard to update your sample with LiteDb as well, if you want.

@jbogard
Copy link
Contributor Author

jbogard commented Jan 11, 2023

Nice! Yeah I was adding the OTel stuff specifically for the demo I was doing there. Thanks!

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 a pull request may close this issue.

2 participants