Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add me change password #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

JonasSyrstad
Copy link
Contributor

In a project im working on I need to change the password of the signed-in user.
https://msdn.microsoft.com/en-us/library/azure/ad/graph/api/functions-and-actions?f=255&MSPPError=-2147217396#changePassword

Thanx

/Jonas

@nianton
Copy link
Owner

nianton commented Dec 2, 2018

Hello Jonas,

Could it be achieved via using the existing method UserResetPasswordAsync? I think the "me" endpoint will not be available when the client is set to use application credentials (appId and appSecret), which will complicate the expected behavior for other use cases.

@JonasSyrstad
Copy link
Contributor Author

There are two problems with UserResetPasswordAsync:

  1. the "service account" needs to much privileges¨
  2. no way of verifuing the users old password

You could expose the endpoins from Azure Graph that acts on the logged in user in an extension package. This way the base package will be consistent.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants