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

Resolve-DbaNetworName is not working properly in PS6.0+ #4840

Closed
4 tasks done
nvarscar opened this issue Dec 18, 2018 · 2 comments · Fixed by #4870
Closed
4 tasks done

Resolve-DbaNetworName is not working properly in PS6.0+ #4840

nvarscar opened this issue Dec 18, 2018 · 2 comments · Fixed by #4870

Comments

@nvarscar
Copy link
Contributor

Before submitting a bug report:

  • Ensure you are able to reproduce it on the latest released version (we release often)
  • Verified this bug is not already reported in an issue
  • Verified errors are not related to permissions
  • Can reproduce in a clean PowerShell session (clean = powershell -NoProfile)

Steps to Reproduce

$t = Resolve-DbaNetworName -ComputerName computer -Verbose

image

image

Expected Behavior

Normal mode with CIM/PSRemoting hostname verifications.
No extra output.

Actual Behavior

Turbo mode (due to different parameter set in PS6.0 version of Test-Connection).
Ping results show up on the screen.

Environmental data

  • PowerShell:
    PSVersion 6.1.0
    PSEdition Core
    GitCommitId 6.1.0
    OS Microsoft Windows 10.0.15063
    Platform Win32NT
    PSCompatibleVersions {1.0, 2.0, 3.0, 4.0...}
    PSRemotingProtocolVersion 2.3
    SerializationVersion 1.1.0.1
    WSManStackVersion 3.0
@nvarscar
Copy link
Contributor Author

Also, according to search, Test-Connection is used in multiple places. Might be worth replacing with [System.Net.NetworkInformation.Ping]::new().Send($computer)

image

@potatoqualitee
Copy link
Member

go for it

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

Successfully merging a pull request may close this issue.

2 participants