search

microsoftgraph / aspnetcore-connect-sample

[ARCHIVED] This ASP.NET Core MVC sample shows how to connect to Microsoft Graph using delegated permissions and the Azure AD v2.0 (MSAL) endpoint.
MIT License
123 stars 96 forks source link

The Sample works fine with Work or School Account. WHat goes wrong with personal account #12

Closed gerritjb closed 6 years ago

gerritjb commented 6 years ago

When singing in with personal account, and then want to send a mail I get the error page. Even in development mode I don't the more debug info on what goes wrong. Why do I get the /home/error page instead of the Development Error handler. And why can I not send mail with the sample from a personal account?

mark-szabo commented 6 years ago

Thanks for the feedback gerritjb! Will look into this soon. It can be that the email sending endpoint does not support MSA. Or only the v2.0 endpoint supports it.

mark-szabo commented 6 years ago

Yes, the problem is with the endpoint as I thought. When sending an email the user profile picture is attached. Therefore it is needed to be downloaded from Graph first, but it will throw an error saying operation is not supported as getting user picture with an MSA account is only supported with the v2.0 endpoint. Working on a fix now.

mark-szabo commented 6 years ago

Issue should be solved. Let me know if it's working for you and we can close the issue! :)

gerritjb commented 6 years ago

Issue can be closed ;-) Your answer helpt me where to look. ThankS!

mark-szabo commented 6 years ago

Great, check out the commit linked above (a9a7fea) with my fix if you need help with the implementation :)

gerritjb commented 6 years ago

Nice! With the conditional beta endpoint!

From: Mark Szabo notifications@github.com Sent: Sunday, April 8, 2018 11:09:22 AM To: microsoftgraph/aspnetcore-connect-sample Cc: Gerrit Jan Bloem; Author Subject: Re: [microsoftgraph/aspnetcore-connect-sample] The Sample works fine with Work or School Account. WHat goes wrong with personal account (#12)

Great, check out the commit linked above (a9a7feahttps://github.com/microsoftgraph/aspnetcore-connect-sample/commit/a9a7feae734042bd23cbce3e6608c128affefa6e) with my fix if you need help with the implementation :)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/microsoftgraph/aspnetcore-connect-sample/issues/12#issuecomment-379534147, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AbXULzk3nz08GoTBW2I6mjBqKqy2BdTAks5tmdPCgaJpZM4TGQuI.