Particular / Workshop

SOA Done Right
Apache License 2.0
583 stars 224 forks source link

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

Closed jbogard closed 1 year ago

jbogard commented 4 years ago

About time for an upgrade I think :)

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

jbogard commented 4 years ago

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 commented 4 years ago

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 commented 4 years ago

@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 commented 4 years ago

Ah thanks! One less thing to worry about

On Mon, Jan 27, 2020 at 7:03 AM Dennis van der Stelt < notifications@github.com> wrote:

@jbogard https://github.com/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.

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/Particular/Workshop/issues/323?email_source=notifications&email_token=AAAZQMXEW4SQAPDTP22HVQDQ73LRLA5CNFSM4JOQR4QKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJ7NEEI#issuecomment-578736657, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAZQMQQUMQ3CQPSDXX63WTQ73LRLANCNFSM4JOQR4QA .

jbogard commented 2 years ago

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 commented 2 years ago

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

dvdstelt commented 1 year ago

@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 commented 1 year ago

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