search

TheJumpCloud / jumpcloud-ADMU

JumpCloud Active Directory Migration Utility - JCADMU
23 stars 7 forks source link

Sa 3327 support for local user account #92

Closed kmaranionjc closed 1 year ago

kmaranionjc commented 1 year ago

Issues

What does this solve?

This change fixes bug and allows admins to migrate a JC user with a Local User Account

Is there anything particularly tricky?

N/A

How should this be tested?

  1. Form:

  1. CMD:

    • Run the test using with a user that has local user account set and autobind
    • Validate that the user has been successfully migrated and should have the system/computer account name = JC SystemUsername

  2. Run migration with a JC user that have no local user account and autobind = $true

    • Validate it is working properly

  3. Run migration with a JC user that have no local user account and autobind = $false

    • Validate it is working properly

Screenshots

image

After migration:

image
jworkmanjc commented 1 year ago

Here's how I'm testing.

Form Manual Tests:

✅ Test works as expected. Screen Shot 2023-05-30 at 12 38 58 PM Screen Shot 2023-05-30 at 12 41 53 PM

CMD tests: Run the test using with a user that has local user account set and autobind Validate that the user has been successfully migrated and should have the system/computer account name = JC SystemUsername Run migration with a JC user that have no local user account and autobind = $true Validate it is working properly Run migration with a JC user that have no local user account and autobind = $false

Automated tests account for this but in my testing this will also work ✅

Lastly, we updated form.exe with password boxes. Both the values from the connectKey & APIKey should pass their respective values to Start-Migration when used