Azure / Azure/azure-libraries-for-java

[FEATURE REQ] Check subscription id early and provide more friendly error message

Open
#945 1 comment 0 reactions 1 assignee Claimed by @weidongxu-microsoft View on GitHub
Authentication Mgmt
Dominant language
Java
Stars
97
Forks
102
PR merge metrics
No merged PRs in 30d

Description

**Is your feature request related to a problem? Please describe.**

We are building java toolkit based on azure java library, and we found the error message is a bit confusing when subscription is set to null. [Related issue](https://github.com/microsoft/azure-maven-plugins/issues/931). This could be reproduced when you auth with an Microsoft account without any subscription

**Describe the solution you'd like**
We could throw specific kind of Exception(like 'AzureAuthException') and show more friendly error message("Subscription Id is not specified"), or we could set error code to indicate this is an auth issue.

**Describe alternatives you've considered**
We could check subscription early, like in the auth stage
- We could check whether specified subs is exist and throw correspond exception in method [withSubscription](https://github.com/Azure/azure-libraries-for-java/blob/master/azure/src/main/java/com/microsoft/azure/management/Azure.java#L394)
- We could check whether there are available subscription in current account in method [withDefaultSubscription](https://github.com/Azure/azure-libraries-for-java/blob/master/azure/src/main/java/com/microsoft/azure/management/Azure.java#L399)

**Information Checklist**
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report
- [x] Description Added
- [x] Expected solution specified

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.