Add additional parameter validation before authentication

Azure / Synapse-workspace-deployment

MIT License

27 stars 36 forks source link

Add additional parameter validation before authentication #31

Closed cpc-camarj closed 2 years ago

cpc-camarj commented 2 years ago

Hello,

I was switching from using a managed identity in one subscription to service principals in multiple subscriptions and spent a long time debugging the problem. This PR includes the fixes that I think will help reduce the amount of time others spend debugging.

Adds additional parameter validation.
Adds debug logging to help with determining where a failure is occurring during authentication
Adds ncc as an additional dev dependency to help with generating dist/index.js

related to #30

ghost commented 2 years ago

All CLA requirements met.

cpc-camarj commented 2 years ago

Lots of changes to upstream since this was originally submitted. I'll go ahead and close this since it probably introduced too much or was headed in the wrong direction.