sailpoint-oss / golang-sdk

MIT License
9 stars 3 forks source link

NewDefaultConfiguration() should return an error instead of calling panic() #70

Open aogail opened 2 months ago

aogail commented 2 months ago

I am exploring using the Sailpoint Go SDK and discovered that sailpoint.NewDefaultConfiguration() panics if it can't find a configuration. I would prefer it to return an error instead of calling panic, so that higher level application logic can recognize the problem and report it appropriately to the user.

See https://github.com/sailpoint-oss/golang-sdk/blob/976088a5b0efb0c1ff5a5605b0016abd7304afa1/configuration.go#L194