Extend the `m365 status` command with appId and tenant

We should extend the output of the m365 status command with information about the appId and tenant used in authentication.

We should extend the output from:

{
  "connectedAs": "name"
}

to:

{
  "connectedAs": "name",
  "appId": "123",
  "appTenant": "common"
}

Rationale

After experimenting with a custom AAD app, today, while trying to create an app reg using the CLI, I got an “Insufficient privileges to complete the operation” error. After scratching my head for a sec I realized, that it’s because the app reg I was using yesterday had no scopes for managing AAD apps. That’s not obvious when you check status, because it only shows the user or app name. Unless you know to check the token, it’s not trivial to figure out and is something that we should communicate more clearly.

[waldekmastykarz]

Yeah exactly!

What if we used not isAppOnlyConnected but rather show the value of the authType option that was used during login? It would not only allow you to determine if you’re using app-only or not but could possibly support other scenarios as well.

Not sure what you mean here. Is the tenant name stored somewhere while logging into the CLI?

Yes, we store the information along tokens. We’re serializing the instance of the Service class and have access to its properties https://github.com/pnp/cli-microsoft365/blob/1e08e6ce84efcbb4a65f1c96998588a5b3420a98/src/Auth.ts#L37-L51. You can get to it via auth.service.

[milanholemans]

I think it’s a great and useful addition to always show 😊