Azure / Azure/azure-libraries-for-java

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

Aperta
#945 1 commento 0 reazioni 1 assegnatario Rivendicata da @weidongxu-microsoft Vedi su GitHub
Authentication Mgmt
Lingua principale
Java
Stelle
97
Fork
102
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

**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

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.