search

Brightspace / D2L.Security.OAuth2

Brightspace OAuth 2.0 for C#
Apache License 2.0
7 stars 16 forks source link

Remove some .NET Framework references #233

Closed cesar-d2l closed 2 years ago

cesar-d2l commented 2 years ago

This removes System.IdentityModel which wasn't being referenced anywhere

The motivation for doing this: We want to remove System.IdentityModel from the LMS (well all References) in our transition to .NET. However I have been noticing certain projects building that use classes in System.IdentityModel but don't reference System.IdentityModel. We think it's because it's picking up System.IdentityModel as a transitive dependency. For example, D2L.LP.Auth depends on this package as well as System.IdentityModel. Even though I remove System.IdentityModel reference, the project still compiles :(

cesar-d2l commented 2 years ago

Can anyone with write access merge this in?

j3parker commented 2 years ago

Thanks!