-
Notifications
You must be signed in to change notification settings - Fork 148
Update WS1-ReEnroll.ps1 #253
base: master
Are you sure you want to change the base?
Conversation
It appears that VMware blocks ICMP so if we just want to check that the server address is correct and responds then maybe using TCP and port 80 is a better option.
@BWMerlin, we have received your signed contributor license agreement. It will be reviewed by VMware shortly. Another comment will be added to the pull request to notify you when the merge can proceed. |
@BWMerlin, we have received your signed contributor license agreement. It will be reviewed by VMware shortly. Another comment will be added to the pull request to notify you when the merge can proceed. |
Test-Connection can fail as ICMP can be blocked. Due to PowerShell 5.x limitations -TcpPort is only available PS7.x so we cannot specify a port as it may not work when using PS5.x which is the minimum version we can expect in production. We can get around this by using Test-NetConnection which gives us access to -Port where we can use port 443 as port 80 is blocked in my testing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed test-connection to test-netconnection
@BWMerlin, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed. |
@BWMerlin, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed. |
@BWMerlin, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed. |
@BWMerlin, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed. |
@BWMerlin, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed. |
@BWMerlin, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed. |
@BWMerlin, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed. |
@BWMerlin, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed. |
@BWMerlin, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed. |
It appears that VMware blocks ICMP so if we just want to check that the server address is correct and responds then maybe using TCP and port 80 is a better option.