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

SetupAdminAccount.aspx & OWIN customizations #38

Closed
yemoku opened this issue Jun 29, 2015 · 6 comments
Closed

SetupAdminAccount.aspx & OWIN customizations #38

yemoku opened this issue Jun 29, 2015 · 6 comments
Assignees
Milestone

Comments

@yemoku
Copy link

yemoku commented Jun 29, 2015

Noticed this behaviour in both releases. If you make any changes to the default OWIN behaviour whilst setting up a new project, these changes are not reflected at runtime.

Example:
In the ApplicationUserManager Create method, make a change to manager.PasswordValidator by setting RequiredLength = 5, OWIN will still prompt the user to create a password of minimum of 6 characters

@fschultz
Copy link
Contributor

fschultz commented Jul 1, 2015 via email

@fschultz fschultz added this to the Release 1.0.1 milestone Aug 20, 2015
@fschultz fschultz self-assigned this Aug 20, 2015
@fschultz
Copy link
Contributor

Sorry for the late response. The problem is that the Owin context isn't used but instead a default UserManager is created when users are created, both in the initial setup page but also within the admin UI. I will fix this in the next version (1.1.0)

@fschultz
Copy link
Contributor

Besides reference the updated KalikoCMS.Identity existing projects need to update the following files:
IdentityConfig.cs dc4eb10#diff-8350f2838377b82e7e14e4d10c31117a

Startup.Auth.cs dc4eb10#diff-ef7c0a70ac9892ac1d2bfb873a7d70a8

SetupAdminAccount.aspx dc4eb10#diff-46a6112a12e06cbe2d9ce23d51e5a2ab

@yemoku
Copy link
Author

yemoku commented Sep 13, 2015

Great stuff! Will download and give feedback.

@yemoku
Copy link
Author

yemoku commented Sep 14, 2015

Cool. It works. Seeing as you hadnt released any code I had to "hack around a bit". Here's what I did:

  1. Created a new project and used Nuget to get all the relevant references
  2. Made code changes as per your instructions
  3. Downloaded your Kaliko.Identity Source and referenced in my solution, removing the compiled Nuget version and removed the duplicate startup.cs in the downloaded source
  4. Changed the password validator code to do a minimum of 4 chars for a new password
  5. Logged in as Admin and created a new user with 4 chars for a new password

You can close out this entry once you ship your updated 1.1.0 version (or mark as fixed for that version if you wish)

@fschultz
Copy link
Contributor

Thanks for the feedback! I hope to have 1.1.0 ready pretty soon if all goes well.

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

2 participants