okta / samples-blazor

samples-blazor
https://github.com/okta/samples-blazor
Apache License 2.0
16 stars 19 forks source link

Nickdrouin/net80 #20

Open NickDrouin opened 6 months ago

NickDrouin commented 6 months ago

I created a new project using the latest net8.0 tooling for server-side Blazor. As such, many of the pages moved around within the repo, with no significant functional changes.

Note: The Weather controller now forces Authorization, I felt this wasn't demonstrated in the original sample and is a common use-case for devs.

The Program.cs both has changes to match the latest dotnet new template, but also has a work-around for: https://github.com/okta/okta-aspnet/issues/260 This commit should be dropped once the issue is fixed in the okta-aspnet lib. Meanwhile, it can be considered by other devs as a work-around.

davidcorbin-atmosera commented 6 months ago

@NickDrouin - with 4.6.1 now released, does it make sense to update this?

NickDrouin commented 6 months ago

@NickDrouin - with 4.6.1 now released, does it make sense to update this?

Thanks @davidcorbin-atmosera , I updated the lib and removed the work-around.