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

ChangePasswordMenuItem should not appear for Trusted Provider #148

Open
ccoulson opened this issue Oct 31, 2017 · 2 comments
Open

ChangePasswordMenuItem should not appear for Trusted Provider #148

ccoulson opened this issue Oct 31, 2017 · 2 comments

Comments

@ccoulson
Copy link
Member

lancegosby[CodePlex]
My web application allows users to authenticate with FBA as well as ADFS. When logged in with an FBA user, I see the quotChange Passwordquot menu item in the Personal Actions menu as expected. When logged in as an ADFS user, I still see quotChange
Passwordquot but didn't expect to.

In ChangePasswordMenuItem.cs, it checks for the type of issuer and only returns early if the type is quotWindowsquot. Perhaps it would be better to return if the type is NOT quotFormsquot.

@ccoulson
Copy link
Member Author

ccoulson[CodePlex]
Thanks for the submission! I'll include it in the next release.

@ccoulson
Copy link
Member Author

lancegosby[CodePlex]
I've made the following change to ChangePasswordMenuItem.cs in my environment and it seems to work now as expected with Forms, ADFS (TrustedProvider) and Windows.

Line 25 of ChangePasswordMenuItem.cs:

Old:
if (SPOriginalIssuers.GetIssuerType(sPClaim.OriginalIssuer) == SPOriginalIssuerType.Windows)
New:
if (SPOriginalIssuers.GetIssuerType(sPClaim.OriginalIssuer) != SPOriginalIssuerType.Forms)
Sorry I'm not sure how to submit a Pull Request on CodePlex.

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

No branches or pull requests

1 participant