jprdonnelly/opentf
1
0
Fork 0
mirror of https://github.com/opentffoundation/opentf.git synced 2026-03-23 13:00:40 -04:00
Code Issues Packages Projects Releases Wiki Activity

provider/azurerm: Remove editorialization from comment

Browse Source
This commit is contained in:
Paul Hinze
2016-09-05 15:02:27 -05:00
parent 6de78da03d
commit 0db012fcf1
1 changed files with 1 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
builtin/providers/azurerm/config.go
View File

@@ -135,12 +135,7 @@ func (c *Config) getArmClient() (*ArmClient, error) {
return nil, err
}
// This is necessary because no-one thought about API usability. OAuthConfigForTenant
// returns a pointer, which can be nil. NewServicePrincipalToken does not take a pointer.
// Consequently we have to nil check this and do _something_ if it is nil, which should
// be either an invariant of OAuthConfigForTenant (guarantee the token is not nil if
// there is no error), or NewServicePrincipalToken should error out if the configuration
// is required and is nil. This is the worst of all worlds, however.
// OAuthConfigForTenant returns a pointer, which can be nil.
if oauthConfig == nil {
return nil, fmt.Errorf("Unable to configure OAuthConfig for tenant %s", c.TenantID)
}