AzureAD / MSAL.PS

MIT License
159 stars 29 forks source link

Object check for $ClientApplication.AppConfig substructure options #52

Closed mortenlerudjordet closed 2 years ago

mortenlerudjordet commented 2 years ago

Possible fix for #45

Added check on $ClientApplication.AppConfig to handle substructure change for AuthorityInfo.Host or the old Authority.AuthorityInfo.Host.

mortenlerudjordet commented 2 years ago

Not sure why build fails, when I run it in my AzDo build goes through.

And I cant see the error message the build fails on, so I could find a fix for it.

mortenlerudjordet commented 2 years ago

@jasoth : Is this PR something you would consider, or should i just abandon it?

jazuntee commented 2 years ago

@mortenlerudjordet thanks for the PR. Unfortunately, this issue is likely caused by a DLL version conflict in your PowerShell session so making the change in this PR would break anyone without the DLL conflict. See my response on the issue. https://github.com/AzureAD/MSAL.PS/issues/45#issuecomment-1170009721