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

Users/vinca/bugfix #2053

Merged
merged 8 commits into from
Jul 12, 2016
Merged

Users/vinca/bugfix #2053

merged 8 commits into from
Jul 12, 2016

Conversation

vincent1173
Copy link
Contributor

Replacing ".\username" by "MachineName\username".
Tested with both domain and workgroup.
Patch Version Incremented.
Gulp tests passed.

@@ -75,7 +75,19 @@ param (
ThrowError -errorMessage (Get-LocalizedString -Key "Parameter '{0}' cannot be null or empty." -ArgumentList "credential")
}
}


function Get-FullUserName(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you name this function Get-DownLevelLogonName... Refer: https://msdn.microsoft.com/en-us/library/windows/desktop/aa380525(v=vs.85).aspx.

@kmkumaran
Copy link
Member

Looks fine.

@KrishnaAdityaB
Copy link
Contributor

Vincent,
I think these code changes don't support the case where FQDN of the machine is given as - mymachinename.fareast.corp.microsoft.com. Can you quickly verify?

@vincent1173
Copy link
Contributor Author

Hi Krishna,
Domain and Local user credentials tests passed.
Gulp tests passed

@vincent1173 vincent1173 merged commit b525bec into master Jul 12, 2016
@bryanmacfarlane bryanmacfarlane deleted the users/vinca/bugfix branch October 20, 2016 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants