[Bug] Need one more err check and defer

AzureAD / microsoft-authentication-library-for-go

The MSAL library for Go is part of the Microsoft identity platform for developers (formerly named Azure AD) v2.0. It enables you to acquire security tokens to call protected APIs. It uses industry standard OAuth2 and OpenID Connect.

MIT License

232 stars 88 forks source link

[Bug] Need one more err check and defer #487

Open TheRealMal opened 5 months ago

TheRealMal commented 5 months ago

Another err check should be placed with close defer https://github.com/AzureAD/microsoft-authentication-library-for-go/blob/882b562b72d131d87e866691c54cbc08111ba804/apps/internal/oauth/ops/authority/authority.go#L553-L558 Like here https://github.com/AzureAD/microsoft-authentication-library-for-go/blob/882b562b72d131d87e866691c54cbc08111ba804/apps/internal/oauth/ops/authority/authority.go#L548-L551

bgavrilMS commented 2 months ago

Sorry for the late reply. Can you please describe how this inconsistency affects an app using MSAL Go?